chiark / gitweb /
journal: PAGE_SIZE is not known on ppc and other archs
authorLennart Poettering <lennart@poettering.net>
Wed, 21 Mar 2012 22:47:44 +0000 (23:47 +0100)
committerLennart Poettering <lennart@poettering.net>
Wed, 21 Mar 2012 22:47:44 +0000 (23:47 +0100)
Let's use NAME_MAX, as suggested by Dan Walsh

src/journal/journald.c

index d27cb60ac7e7791b137c4c9a7c44859e544272dd..87390bdec9921538e3f4c2f6943fffab79e25703 100644 (file)
@@ -29,7 +29,6 @@
 #include <sys/ioctl.h>
 #include <linux/sockios.h>
 #include <sys/statvfs.h>
-#include <sys/user.h>
 
 #include <systemd/sd-journal.h>
 #include <systemd/sd-login.h>
@@ -2149,10 +2148,20 @@ static int process_event(Server *s, struct epoll_event *ev) {
                         size_t label_len = 0;
                         union {
                                 struct cmsghdr cmsghdr;
+
+                                /* We use NAME_MAX space for the
+                                 * SELinux label here. The kernel
+                                 * currently enforces no limit, but
+                                 * according to suggestions from the
+                                 * SELinux people this will change and
+                                 * it will probably be identical to
+                                 * NAME_MAX. For now we use that, but
+                                 * this should be updated one day when
+                                 * the final limit is known.*/
                                 uint8_t buf[CMSG_SPACE(sizeof(struct ucred)) +
                                             CMSG_SPACE(sizeof(struct timeval)) +
-                                            CMSG_SPACE(sizeof(int)) +
-                                            CMSG_SPACE(PAGE_SIZE)]; /* selinux label */
+                                            CMSG_SPACE(sizeof(int)) + /* fd */
+                                            CMSG_SPACE(NAME_MAX)]; /* selinux label */
                         } control;
                         ssize_t n;
                         int v;