diff options
author | Lennart Poettering <lennart@poettering.net> | 2012-11-20 21:25:26 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2012-11-20 21:39:16 +0100 |
commit | ee55db41442ad8055f5a84a339b1e0e22bc037c4 (patch) | |
tree | 2bfb25c91d53eebce0c7cb8a276d267612f95563 /src | |
parent | 3ed08c446cfaaae2b234fdfeb0c34ab6b4748c3e (diff) |
journal-send: always send SYSLOG_IDENTIFIER, if we have it
https://bugzilla.redhat.com/show_bug.cgi?id=872193
Diffstat (limited to 'src')
-rw-r--r-- | src/journal/journal-send.c | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/src/journal/journal-send.c b/src/journal/journal-send.c index 7a91569055..bd8f8872c9 100644 --- a/src/journal/journal-send.c +++ b/src/journal/journal-send.c @@ -219,6 +219,7 @@ _public_ int sd_journal_sendv(const struct iovec *iov, int n) { * be a tmpfs, and one that is available from early boot on * and where unprivileged users can create files. */ char path[] = "/dev/shm/journal.XXXXXX"; + bool have_syslog_identifier = false; if (_unlikely_(!iov)) return -EINVAL; @@ -228,7 +229,7 @@ _public_ int sd_journal_sendv(const struct iovec *iov, int n) { saved_errno = errno; - w = alloca(sizeof(struct iovec) * n * 5); + w = alloca(sizeof(struct iovec) * n * 5 + 3); l = alloca(sizeof(uint64_t) * n); for (i = 0; i < n; i++) { @@ -245,6 +246,9 @@ _public_ int sd_journal_sendv(const struct iovec *iov, int n) { goto finish; } + have_syslog_identifier = + have_syslog_identifier || (c == iov[i].iov_base + 17 && memcmp(iov[i].iov_base, "SYSLOG_IDENTIFIER", 17) == 0); + nl = memchr(iov[i].iov_base, '\n', iov[i].iov_len); if (nl) { if (_unlikely_(nl < c)) { @@ -280,6 +284,20 @@ _public_ int sd_journal_sendv(const struct iovec *iov, int n) { IOVEC_SET_STRING(w[j++], "\n"); } + if (!have_syslog_identifier && + string_is_safe(program_invocation_short_name)) { + + /* Implicitly add program_invocation_short_name, if it + * is not set explicitly. We only do this for + * program_invocation_short_name, and nothing else + * since everything else is much nicer to retrieve + * from the outside. */ + + IOVEC_SET_STRING(w[j++], "SYSLOG_IDENTIFIER="); + IOVEC_SET_STRING(w[j++], program_invocation_short_name); + IOVEC_SET_STRING(w[j++], "\n"); + } + fd = journal_fd(); if (_unlikely_(fd < 0)) { r = fd; |