journal: remove template from open_journal args
None of the callers take advantage of this parameter, it's always NULL, this is just a private helper function to simplify the call sites so drop the template parameter altogether. If a caller emerges later who needs it, it can be restored.
This commit is contained in:
@@ -244,7 +244,6 @@ static int open_journal(
|
||||
int flags,
|
||||
bool seal,
|
||||
JournalMetrics *metrics,
|
||||
JournalFile *template,
|
||||
JournalFile **ret) {
|
||||
int r;
|
||||
JournalFile *f;
|
||||
@@ -254,9 +253,9 @@ static int open_journal(
|
||||
assert(ret);
|
||||
|
||||
if (reliably)
|
||||
r = journal_file_open_reliably(fname, flags, 0640, s->compress, seal, metrics, s->mmap, template, &f);
|
||||
r = journal_file_open_reliably(fname, flags, 0640, s->compress, seal, metrics, s->mmap, NULL, &f);
|
||||
else
|
||||
r = journal_file_open(fname, flags, 0640, s->compress, seal, metrics, s->mmap, template, &f);
|
||||
r = journal_file_open(fname, flags, 0640, s->compress, seal, metrics, s->mmap, NULL, &f);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
@@ -308,7 +307,7 @@ static JournalFile* find_journal(Server *s, uid_t uid) {
|
||||
journal_file_close(f);
|
||||
}
|
||||
|
||||
r = open_journal(s, true, p, O_RDWR|O_CREAT, s->seal, &s->system_metrics, NULL, &f);
|
||||
r = open_journal(s, true, p, O_RDWR|O_CREAT, s->seal, &s->system_metrics, &f);
|
||||
if (r < 0)
|
||||
return s->system_journal;
|
||||
|
||||
@@ -1000,7 +999,7 @@ static int system_journal_open(Server *s, bool flush_requested) {
|
||||
(void) mkdir(fn, 0755);
|
||||
|
||||
fn = strjoina(fn, "/system.journal");
|
||||
r = open_journal(s, true, fn, O_RDWR|O_CREAT, s->seal, &s->system_metrics, NULL, &s->system_journal);
|
||||
r = open_journal(s, true, fn, O_RDWR|O_CREAT, s->seal, &s->system_metrics, &s->system_journal);
|
||||
if (r >= 0) {
|
||||
server_add_acls(s->system_journal, 0);
|
||||
(void) determine_space_for(s, &s->system_metrics, "/var/log/journal/", "System journal", true, true, NULL, NULL);
|
||||
@@ -1023,7 +1022,7 @@ static int system_journal_open(Server *s, bool flush_requested) {
|
||||
* if it already exists, so that we can flush
|
||||
* it into the system journal */
|
||||
|
||||
r = open_journal(s, false, fn, O_RDWR, false, &s->runtime_metrics, NULL, &s->runtime_journal);
|
||||
r = open_journal(s, false, fn, O_RDWR, false, &s->runtime_metrics, &s->runtime_journal);
|
||||
if (r < 0) {
|
||||
if (r != -ENOENT)
|
||||
log_warning_errno(r, "Failed to open runtime journal: %m");
|
||||
@@ -1040,7 +1039,7 @@ static int system_journal_open(Server *s, bool flush_requested) {
|
||||
(void) mkdir("/run/log/journal", 0755);
|
||||
(void) mkdir_parents(fn, 0750);
|
||||
|
||||
r = open_journal(s, true, fn, O_RDWR|O_CREAT, false, &s->runtime_metrics, NULL, &s->runtime_journal);
|
||||
r = open_journal(s, true, fn, O_RDWR|O_CREAT, false, &s->runtime_metrics, &s->runtime_journal);
|
||||
if (r < 0)
|
||||
return log_error_errno(r, "Failed to open runtime journal: %m");
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user