From 5e41cfec83aa47af12e469bc62e336f8213ee066 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Wed, 11 Jan 2012 21:11:58 +0100 Subject: [PATCH] journald: set group ownership of journal files to 'adm' by default --- src/journal/journald.c | 31 ++++++++++++++++++++++++++----- src/journal/journald.h | 3 +++ 2 files changed, 29 insertions(+), 5 deletions(-) diff --git a/src/journal/journald.c b/src/journal/journald.c index 0194a1b8..33865b81 100644 --- a/src/journal/journald.c +++ b/src/journal/journald.c @@ -183,7 +183,26 @@ finish: return avail; } -static void fix_perms(JournalFile *f, uid_t uid) { +static void server_read_file_gid(Server *s) { + const char *adm = "adm"; + int r; + + assert(s); + + if (s->file_gid_valid) + return; + + r = get_group_creds(&adm, &s->file_gid); + if (r < 0) + log_warning("Failed to resolve 'adm' group: %s", strerror(-r)); + + /* if we couldn't read the gid, then it will be 0, but that's + * fine and we shouldn't try to resolve the group again, so + * let's just pretend it worked right-away. */ + s->file_gid_valid = true; +} + +static void server_fix_perms(Server *s, JournalFile *f, uid_t uid) { int r; #ifdef HAVE_ACL acl_t acl; @@ -193,7 +212,9 @@ static void fix_perms(JournalFile *f, uid_t uid) { assert(f); - r = fchmod_and_fchown(f->fd, 0640, 0, 0); + server_read_file_gid(s); + + r = fchmod_and_fchown(f->fd, 0640, 0, s->file_gid); if (r < 0) log_warning("Failed to fix access mode/rights on %s, ignoring: %s", f->path, strerror(-r)); @@ -277,7 +298,7 @@ static JournalFile* find_journal(Server *s, uid_t uid) { if (r < 0) return s->system_journal; - fix_perms(f, uid); + server_fix_perms(s, f, uid); f->metrics = s->system_metrics; f->compress = s->compress; @@ -1733,7 +1754,7 @@ static int system_journal_open(Server *s) { s->system_journal->metrics = s->system_metrics; s->system_journal->compress = s->compress; - fix_perms(s->system_journal, 0); + server_fix_perms(s, s->system_journal, 0); } else if (r < 0) { if (r != -ENOENT && r != -EROFS) @@ -1786,7 +1807,7 @@ static int system_journal_open(Server *s) { s->runtime_journal->metrics = s->runtime_metrics; s->runtime_journal->compress = s->compress; - fix_perms(s->runtime_journal, 0); + server_fix_perms(s, s->runtime_journal, 0); } } diff --git a/src/journal/journald.h b/src/journal/journald.h index 1f1665bf..6160991e 100644 --- a/src/journal/journald.h +++ b/src/journal/journald.h @@ -73,6 +73,9 @@ typedef struct Server { uint64_t var_available_timestamp; + gid_t file_gid; + bool file_gid_valid; + LIST_HEAD(StdoutStream, stdout_streams); unsigned n_stdout_streams; } Server; -- 2.39.5