[systemd-commits] 2 commits - Makefile.am src/kmsg-syslogd.c units/syslog.socket units/systemd-journald.service.in
Lennart Poettering
lennart at kemper.freedesktop.org
Thu Jan 5 18:14:15 PST 2012
Makefile.am | 22 --
src/kmsg-syslogd.c | 417 --------------------------------------
units/syslog.socket | 23 +-
units/systemd-journald.service.in | 3
4 files changed, 20 insertions(+), 445 deletions(-)
New commits:
commit ba1d5596354e3299d5b9b995bb0fa9b3bdd285cc
Author: Lennart Poettering <lennart at poettering.net>
Date: Fri Jan 6 03:11:56 2012 +0100
journald: start the journal after the syslog socket, so that the syslog socket queues syslog messages from early boot on
diff --git a/units/systemd-journald.service.in b/units/systemd-journald.service.in
index 2cfc684..08858f3 100644
--- a/units/systemd-journald.service.in
+++ b/units/systemd-journald.service.in
@@ -12,12 +12,13 @@ Description=Journal Service
DefaultDependencies=no
Requires=systemd-journald.socket
After=systemd-journald.socket
+After=syslog.socket
[Service]
ExecStart=@rootlibexecdir@/systemd-journald
NotifyAccess=all
StandardOutput=null
-#CapabilityBoundingSet=CAP_SYS_ADMIN CAP_SETUID CAP_SETGID
+#CapabilityBoundingSet=CAP_SYS_ADMIN CAP_SETUID CAP_SETGID CAP_DAC_OVERRIDE
# Increase the default a bit in order to allow many simultaneous
# services being run since we keep one fd open per service.
commit 4c0bebc335c279ecaea6fc0eb0acaa8c49c477b6
Author: Lennart Poettering <lennart at poettering.net>
Date: Fri Jan 6 03:11:17 2012 +0100
kmsg-syslogd: remove kmsg-syslogd, since it's entirely obsoleted and replaced by journald
diff --git a/Makefile.am b/Makefile.am
index 0bd893f..bd7eb5c 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -188,7 +188,6 @@ rootlibexec_PROGRAMS = \
systemd-shutdown \
systemd-modules-load \
systemd-remount-api-vfs \
- systemd-kmsg-syslogd \
systemd-reply-password \
systemd-fsck \
systemd-timestamp \
@@ -318,7 +317,6 @@ nodist_systemunit_DATA = \
units/console-shell.service \
units/systemd-initctl.service \
units/systemd-shutdownd.service \
- units/systemd-kmsg-syslogd.service \
units/systemd-modules-load.service \
units/systemd-remount-api-vfs.service \
units/systemd-update-utmp-runlevel.service \
@@ -351,7 +349,6 @@ EXTRA_DIST += \
units/rescue.service.m4 \
units/systemd-initctl.service.in \
units/systemd-shutdownd.service.in \
- units/systemd-kmsg-syslogd.service.in \
units/systemd-modules-load.service.in \
units/systemd-remount-api-vfs.service.in \
units/systemd-update-utmp-runlevel.service.in \
@@ -955,14 +952,6 @@ systemd_cgroups_agent_LDADD = \
libsystemd-basic.la \
$(DBUS_LIBS)
-systemd_kmsg_syslogd_SOURCES = \
- src/kmsg-syslogd.c \
- src/fdset.c
-
-systemd_kmsg_syslogd_LDADD = \
- libsystemd-basic.la \
- libsystemd-daemon.la
-
systemctl_SOURCES = \
src/systemctl.c \
src/utmp-wtmp.c \
@@ -2233,10 +2222,9 @@ systemd-install-data-hook:
rm -f user && \
$(LN_S) $(pkgsysconfdir)/user user )
( cd $(DESTDIR)$(systemunitdir)/sockets.target.wants && \
- rm -f systemd-initctl.socket systemd-shutdownd.socket syslog.socket && \
+ rm -f systemd-initctl.socket systemd-shutdownd.socket && \
$(LN_S) ../systemd-initctl.socket systemd-initctl.socket && \
- $(LN_S) ../systemd-shutdownd.socket systemd-shutdownd.socket && \
- $(LN_S) ../syslog.socket syslog.socket )
+ $(LN_S) ../systemd-shutdownd.socket systemd-shutdownd.socket )
( cd $(DESTDIR)$(systemunitdir)/runlevel1.target.wants && \
rm -f systemd-update-utmp-runlevel.service && \
$(LN_S) ../systemd-update-utmp-runlevel.service systemd-update-utmp-runlevel.service )
@@ -2307,8 +2295,7 @@ systemd-install-data-hook:
systemd-modules-load.service \
systemd-tmpfiles-setup.service \
systemd-sysctl.service \
- systemd-ask-password-console.path \
- systemd-kmsg-syslogd.service && \
+ systemd-ask-password-console.path && \
$(LN_S) ../dev-hugepages.mount dev-hugepages.mount && \
$(LN_S) ../dev-mqueue.mount dev-mqueue.mount && \
$(LN_S) ../sys-kernel-config.mount sys-kernel-config.mount && \
@@ -2318,8 +2305,7 @@ systemd-install-data-hook:
$(LN_S) ../systemd-modules-load.service systemd-modules-load.service && \
$(LN_S) ../systemd-tmpfiles-setup.service systemd-tmpfiles-setup.service && \
$(LN_S) ../systemd-sysctl.service systemd-sysctl.service && \
- $(LN_S) ../systemd-ask-password-console.path systemd-ask-password-console.path && \
- $(LN_S) ../systemd-kmsg-syslogd.service )
+ $(LN_S) ../systemd-ask-password-console.path systemd-ask-password-console.path )
( cd $(DESTDIR)$(systemunitdir)/basic.target.wants && \
rm -f systemd-tmpfiles-clean.timer && \
$(LN_S) ../systemd-tmpfiles-clean.timer systemd-tmpfiles-clean.timer )
diff --git a/src/kmsg-syslogd.c b/src/kmsg-syslogd.c
deleted file mode 100644
index da83bb2..0000000
--- a/src/kmsg-syslogd.c
+++ /dev/null
@@ -1,417 +0,0 @@
-/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/
-
-/***
- This file is part of systemd.
-
- Copyright 2010 Lennart Poettering
-
- systemd is free software; you can redistribute it and/or modify it
- under the terms of the GNU General Public License as published by
- the Free Software Foundation; either version 2 of the License, or
- (at your option) any later version.
-
- systemd is distributed in the hope that it will be useful, but
- WITHOUT ANY WARRANTY; without even the implied warranty of
- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- General Public License for more details.
-
- You should have received a copy of the GNU General Public License
- along with systemd; If not, see <http://www.gnu.org/licenses/>.
-***/
-
-#include <sys/socket.h>
-#include <sys/types.h>
-#include <assert.h>
-#include <time.h>
-#include <string.h>
-#include <stdio.h>
-#include <errno.h>
-#include <unistd.h>
-#include <sys/poll.h>
-#include <sys/epoll.h>
-#include <sys/un.h>
-#include <fcntl.h>
-#include <sys/signalfd.h>
-
-#include <systemd/sd-daemon.h>
-
-#include "util.h"
-#include "log.h"
-#include "fdset.h"
-
-#define SERVER_FD_MAX 16
-
-typedef struct Stream Stream;
-
-typedef struct Server {
- FDSet *syslog_fds;
- int kmsg_fd;
- int epoll_fd;
- int signal_fd;
-} Server;
-
-static void server_done(Server *s) {
- assert(s);
-
- if (s->epoll_fd >= 0)
- close_nointr_nofail(s->epoll_fd);
-
- if (s->kmsg_fd >= 0)
- close_nointr_nofail(s->kmsg_fd);
-
- if (s->signal_fd >= 0)
- close_nointr_nofail(s->signal_fd);
-
- if (s->syslog_fds)
- fdset_free(s->syslog_fds);
-}
-
-static int server_init(Server *s) {
- int i, r, n;
- struct epoll_event ev;
- sigset_t mask;
-
- assert(s);
-
- zero(*s);
- s->kmsg_fd = s->signal_fd = -1;
-
- s->epoll_fd = epoll_create1(EPOLL_CLOEXEC);
- if (s->epoll_fd < 0) {
- log_error("Failed to create epoll object: %m");
- return -errno;
- }
-
- s->syslog_fds = fdset_new();
- if (!s->syslog_fds) {
- log_error("Failed to allocate file descriptor set: %s", strerror(ENOMEM));
- return -ENOMEM;
- }
-
- n = sd_listen_fds(true);
- if (n < 0) {
- log_error("Failed to read listening file descriptors from environment: %s", strerror(-n));
- return n;
- }
-
- if (n <= 0 || n > SERVER_FD_MAX) {
- log_error("No or too many file descriptors passed.");
- return -EINVAL;
- }
-
- for (i = 0; i < n; i++) {
- int fd;
-
- fd = SD_LISTEN_FDS_START+i;
-
- r = sd_is_socket(fd, AF_UNSPEC, SOCK_DGRAM, -1);
- if (r < 0) {
- log_error("Failed to determine file descriptor type: %s", strerror(-r));
- return r;
- }
-
- if (!r) {
- log_error("Wrong file descriptor type.");
- return -EINVAL;
- }
-
- zero(ev);
- ev.events = EPOLLIN;
- ev.data.fd = fd;
- if (epoll_ctl(s->epoll_fd, EPOLL_CTL_ADD, fd, &ev) < 0) {
- log_error("Failed to add server fd to epoll object: %m");
- return -errno;
- }
-
- r = fdset_put(s->syslog_fds, fd);
- if (r < 0) {
- log_error("Failed to store file descriptor in set: %s", strerror(-r));
- return r;
- }
- }
-
- s->kmsg_fd = open("/dev/kmsg", O_WRONLY|O_NOCTTY|O_CLOEXEC);
- if (s->kmsg_fd < 0) {
- log_error("Failed to open /dev/kmsg for logging: %m");
- return -errno;
- }
-
- assert_se(sigemptyset(&mask) == 0);
- sigset_add_many(&mask, SIGINT, SIGTERM, -1);
- assert_se(sigprocmask(SIG_SETMASK, &mask, NULL) == 0);
-
- s->signal_fd = signalfd(-1, &mask, SFD_NONBLOCK|SFD_CLOEXEC);
- if (s->signal_fd < 0) {
- log_error("signalfd(): %m");
- return -errno;
- }
-
- zero(ev);
- ev.events = EPOLLIN;
- ev.data.fd = s->signal_fd;
-
- if (epoll_ctl(s->epoll_fd, EPOLL_CTL_ADD, s->signal_fd, &ev) < 0) {
- log_error("epoll_ctl(): %m");
- return -errno;
- }
-
- return 0;
-}
-
-static int read_process(const char **buf, struct iovec *iovec) {
- const char *p;
- size_t l;
-
- assert(buf);
- assert(*buf);
- assert(iovec);
-
- p = *buf;
-
- p += strspn(p, WHITESPACE);
- l = strcspn(p, WHITESPACE);
-
- if (l <= 0 ||
- p[l-1] != ':')
- return 0;
-
- l--;
-
- if (p[l-1] == ']') {
- size_t k = l-1;
-
- for (;;) {
-
- if (p[k] == '[') {
- l = k;
- break;
- }
-
- if (k == 0)
- break;
-
- k--;
- }
- }
-
- iovec->iov_base = (char*) p;
- iovec->iov_len = l;
- *buf = p + l;
- return 1;
-}
-
-static int write_message(Server *s, const char *buf, struct ucred *ucred) {
- ssize_t k;
- char priority[6], pid[16];
- struct iovec iovec[5];
- unsigned i = 0;
- char *process = NULL;
- int r = 0;
- int prio = LOG_USER | LOG_INFO;
-
- assert(s);
- assert(buf);
-
- parse_syslog_priority((char**) &buf, &prio);
-
- if (*buf == 0)
- return 0;
-
- if ((prio & LOG_FACMASK) == 0)
- prio = LOG_USER | LOG_PRI(prio);
-
- /* First, set priority field */
- snprintf(priority, sizeof(priority), "<%i>", prio);
- char_array_0(priority);
- IOVEC_SET_STRING(iovec[i++], priority);
-
- /* Second, skip date */
- skip_syslog_date((char**) &buf);
-
- /* Then, add process if set */
- if (read_process(&buf, &iovec[i]) > 0)
- i++;
- else if (ucred &&
- ucred->pid > 0 &&
- get_process_comm(ucred->pid, &process) >= 0)
- IOVEC_SET_STRING(iovec[i++], process);
-
- /* Skip the stored PID if we have a better one */
- if (ucred) {
- snprintf(pid, sizeof(pid), "[%lu]: ", (unsigned long) ucred->pid);
- char_array_0(pid);
- IOVEC_SET_STRING(iovec[i++], pid);
-
- skip_syslog_pid((char**) &buf);
-
- if (*buf == ':')
- buf++;
-
- buf += strspn(buf, WHITESPACE);
- }
-
- /* Is the remaining message empty? */
- if (*buf) {
-
- /* And the rest is the message */
- IOVEC_SET_STRING(iovec[i++], buf);
- IOVEC_SET_STRING(iovec[i++], "\n");
-
- if ((k = writev(s->kmsg_fd, iovec, i)) <= 0) {
- log_error("Failed to write log message to kmsg: %s", k < 0 ? strerror(errno) : "short write");
- r = k < 0 ? -errno : -EIO;
- }
- }
-
- free(process);
-
- return r;
-}
-
-static int process_event(Server *s, struct epoll_event *ev) {
- assert(s);
-
- if (ev->events != EPOLLIN) {
- log_info("Got invalid event from epoll.");
- return -EIO;
- }
-
- if (ev->data.fd == s->signal_fd) {
- struct signalfd_siginfo sfsi;
- ssize_t n;
-
- n = read(s->signal_fd, &sfsi, sizeof(sfsi));
- if (n != sizeof(sfsi)) {
-
- if (n >= 0)
- return -EIO;
-
- if (errno == EINTR || errno == EAGAIN)
- return 0;
-
- return -errno;
- }
-
- log_debug("Received SIG%s", signal_to_string(sfsi.ssi_signo));
- return 0;
-
- } else {
- for (;;) {
- char buf[LINE_MAX+1];
- struct msghdr msghdr;
- struct iovec iovec;
- struct ucred *ucred;
- union {
- struct cmsghdr cmsghdr;
- uint8_t buf[CMSG_SPACE(sizeof(struct ucred))];
- } control;
- ssize_t n;
- int k;
- char *e;
-
- zero(iovec);
- iovec.iov_base = buf;
- iovec.iov_len = sizeof(buf)-1;
-
- zero(control);
- zero(msghdr);
- msghdr.msg_iov = &iovec;
- msghdr.msg_iovlen = 1;
- msghdr.msg_control = &control;
- msghdr.msg_controllen = sizeof(control);
-
- n = recvmsg(ev->data.fd, &msghdr, MSG_DONTWAIT);
- if (n < 0) {
-
- if (errno == EINTR || errno == EAGAIN)
- return 1;
-
- log_error("recvmsg() failed: %m");
- return -errno;
- }
-
- if (msghdr.msg_controllen >= CMSG_LEN(sizeof(struct ucred)) &&
- control.cmsghdr.cmsg_level == SOL_SOCKET &&
- control.cmsghdr.cmsg_type == SCM_CREDENTIALS &&
- control.cmsghdr.cmsg_len == CMSG_LEN(sizeof(struct ucred)))
- ucred = (struct ucred*) CMSG_DATA(&control.cmsghdr);
- else
- ucred = NULL;
-
- e = memchr(buf, '\n', n);
- if (e)
- *e = 0;
- else
- buf[n] = 0;
-
- k = write_message(s, strstrip(buf), ucred);
- if (k < 0)
- return k;
- }
- }
-
- return 1;
-}
-
-int main(int argc, char *argv[]) {
- Server server;
- int r;
-
- if (getppid() != 1) {
- log_error("This program should be invoked by init only.");
- return EXIT_FAILURE;
- }
-
- if (argc > 1) {
- log_error("This program does not take arguments.");
- return EXIT_FAILURE;
- }
-
- log_set_target(LOG_TARGET_KMSG);
- log_parse_environment();
- log_open();
-
- umask(0022);
-
- r = server_init(&server);
- if (r < 0)
- goto finish;
-
- log_debug("systemd-kmsg-syslogd running as pid %lu", (unsigned long) getpid());
-
- sd_notify(false,
- "READY=1\n"
- "STATUS=Processing messages...");
-
- for (;;) {
- struct epoll_event event;
-
- r = epoll_wait(server.epoll_fd, &event, 1, -1);
- if (r < 0) {
-
- if (errno == EINTR)
- continue;
-
- log_error("epoll_wait() failed: %m");
- r = -errno;
- goto finish;
- } else if (r == 0)
- break;
-
- r = process_event(&server, &event);
- if (r < 0)
- goto finish;
- else if (r == 0)
- break;
- }
-
- log_debug("systemd-kmsg-syslogd stopped as pid %lu", (unsigned long) getpid());
-
-finish:
- sd_notify(false,
- "STATUS=Shutting down...");
-
- server_done(&server);
-
- return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;
-}
diff --git a/units/syslog.socket b/units/syslog.socket
index 2d42305..323fa86 100644
--- a/units/syslog.socket
+++ b/units/syslog.socket
@@ -20,12 +20,17 @@ ListenDatagram=/run/systemd/journal/syslog
SocketMode=0666
PassCredentials=yes
-# The service we activate on incoming traffic is
-# systemd-kmsg-syslogd.service. That doesn't mean however, that this
-# is the main syslog daemon in the system. Another syslog
-# implementation (which might be started via on-boot or another
-# non-socket activation) can take over possession of the socket and
-# terminate systemd-kmsg-syslogd. It could also simply replace the
-# socket in the file system, and leave systemd-kmsg-syslogd untouched.
-
-Service=systemd-kmsg-syslogd.service
+# The default syslog implementation should make syslog.service a
+# symlink to itself, so that this socket activates the right actual
+# syslog service.
+#
+# Examples:
+#
+# /etc/systemd/system/syslog.service -> /lib/systemd/system/rsyslog.service
+# /etc/systemd/system/syslog.service -> /lib/systemd/system/syslog-ng.service
+#
+# Best way to achieve that is by adding this to your unit file
+# (i.e. to rsyslog.service or syslog-ng.service):
+#
+# [Install]
+# Alias=syslog.service
More information about the systemd-commits
mailing list