diff options
| author | Michal Sekletár <msekleta@redhat.com> | 2020-02-04 14:23:14 +0100 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2020-02-05 18:34:52 +0100 |
| commit | 28ca867abdb20d0e4ac1901e2ed669cdb41ea3f6 (patch) | |
| tree | 4a3a384e787c4810073064cee0cfc48f432ab7ea /src/journal | |
| parent | c7220ca8025e8dbded36131b23a502d975c45754 (diff) | |
| download | systemd-28ca867abdb20d0e4ac1901e2ed669cdb41ea3f6.tar.gz | |
sd-journal: close journal files that were deleted by journald before we've setup inotify watch
Fixes #14695
Diffstat (limited to 'src/journal')
| -rw-r--r-- | src/journal/journal-file.c | 2 | ||||
| -rw-r--r-- | src/journal/journal-file.h | 1 | ||||
| -rw-r--r-- | src/journal/sd-journal.c | 15 |
3 files changed, 17 insertions, 1 deletions
diff --git a/src/journal/journal-file.c b/src/journal/journal-file.c index 505191999b..bd53635860 100644 --- a/src/journal/journal-file.c +++ b/src/journal/journal-file.c @@ -601,7 +601,7 @@ static int journal_file_verify_header(JournalFile *f) { return 0; } -static int journal_file_fstat(JournalFile *f) { +int journal_file_fstat(JournalFile *f) { int r; assert(f); diff --git a/src/journal/journal-file.h b/src/journal/journal-file.h index 502f1f567d..cf0f7691fb 100644 --- a/src/journal/journal-file.h +++ b/src/journal/journal-file.h @@ -145,6 +145,7 @@ int journal_file_open( int journal_file_set_offline(JournalFile *f, bool wait); bool journal_file_is_offlining(JournalFile *f); JournalFile* journal_file_close(JournalFile *j); +int journal_file_fstat(JournalFile *f); DEFINE_TRIVIAL_CLEANUP_FUNC(JournalFile*, journal_file_close); int journal_file_open_reliably( diff --git a/src/journal/sd-journal.c b/src/journal/sd-journal.c index bced8af3e3..3fa98dfda2 100644 --- a/src/journal/sd-journal.c +++ b/src/journal/sd-journal.c @@ -2661,6 +2661,8 @@ _public_ int sd_journal_wait(sd_journal *j, uint64_t timeout_usec) { assert_return(!journal_pid_changed(j), -ECHILD); if (j->inotify_fd < 0) { + Iterator i; + JournalFile *f; /* This is the first invocation, hence create the * inotify watch */ @@ -2668,6 +2670,19 @@ _public_ int sd_journal_wait(sd_journal *j, uint64_t timeout_usec) { if (r < 0) return r; + /* Server might have done some vacuuming while we weren't watching. + Get rid of the deleted files now so they don't stay around indefinitely. */ + ORDERED_HASHMAP_FOREACH(f, j->files, i) { + r = journal_file_fstat(f); + if (r < 0) { + log_debug_errno(r,"Failed to fstat() journal file '%s' : %m", f->path); + continue; + } + + if (f->last_stat.st_nlink <= 0) + remove_file_real(j, f); + } + /* The journal might have changed since the context * object was created and we weren't watching before, * hence don't wait for anything, and return |