chiark / gitweb /
journal: remove journal_file_object_keep/release functions
authorMichal Schmidt <mschmidt@redhat.com>
Fri, 12 Dec 2014 20:52:18 +0000 (21:52 +0100)
committerMichal Schmidt <mschmidt@redhat.com>
Fri, 12 Dec 2014 23:46:40 +0000 (00:46 +0100)
commit1b8951e5bd9b2bf1722098a861055cae0bb52088
treeb4d7c2e0ddc819d510c5c8565226f11bb7ad6e1f
parentfed67c38e3f1cecf4c0571f5603d47b35bff6576
journal: remove journal_file_object_keep/release functions

The only user is sd_journal_enumerate_unique() and, as explained in
the previous commit (fed67c38e3 "journal: map objects to context set by
caller, not by actual object type"), the use of them there is now
superfluous. Let's remove them.

This reverts major parts of commits:
  ae97089d49 journal: fix access to munmapped memory in
             sd_journal_enumerate_unique
  06cc69d44c sd-journal: fix sd_journal_enumerate_unique skipping values

Tested with an "--enable-debug" build and "journalctl --list-boots".
It gives the expected number of results. Additionally, if I then revert
the previous commit ("journal: map objects to context set by caller, not
to actual object type"), it crashes with SIGSEGV, as expected.
src/journal/journal-file.c
src/journal/journal-file.h
src/journal/journal-verify.c
src/journal/mmap-cache.c
src/journal/mmap-cache.h
src/journal/sd-journal.c
src/journal/test-mmap-cache.c