chiark / gitweb /
journald: close passed fds we cannot make sense of
authorLennart Poettering <lennart@poettering.net>
Sat, 29 Nov 2014 23:51:45 +0000 (00:51 +0100)
committerLennart Poettering <lennart@poettering.net>
Sun, 30 Nov 2014 21:58:55 +0000 (22:58 +0100)
This is mostly likely the audit socket, and we really should close it
if we cannot make sense of it, since as long as it is open the kernel
might disable the kmsg forwarding of audit msgs, and we should avoid
that, since audit msgs might get completely lost then.

I also downgraded the log message we show a bit, after all things should
really work fine, and we proceed fine with it.

src/journal/journald-server.c

index 3bee18b..80c9736 100644 (file)
@@ -1552,8 +1552,16 @@ int server_init(Server *s) {
 
                         s->audit_fd = fd;
 
-                } else
-                        log_error("Unknown socket passed as file descriptor %d, ignoring.", fd);
+                } else {
+                        log_warning("Unknown socket passed as file descriptor %d, ignoring.", fd);
+
+                        /* Let's close the fd, better be safe than
+                           sorry. The fd might reference some resource
+                           that we really want to release if we don't
+                           make use of it. */
+
+                        safe_close(fd);
+                }
         }
 
         r = server_open_syslog_socket(s);