Fix memory leak in stdout journal streams

[elogind.git] / src / journal / journald-stream.c

diff --git a/src/journal/journald-stream.c b/src/journal/journald-stream.c
index 6d51c29083505ef0c0486972b2a04faaee7eef3d..091f59de17c020daa3392e6a906414da2b1cb00a 100644 (file)
--- a/src/journal/journald-stream.c
+++ b/src/journal/journald-stream.c
@@ -29,6 +29,7 @@
 #endif
 
 #include "socket-util.h"
 #endif
 
 #include "socket-util.h"

+#include "selinux-util.h"

 #include "journald-server.h"
 #include "journald-stream.h"
 #include "journald-syslog.h"
 #include "journald-server.h"
 #include "journald-stream.h"
 #include "journald-syslog.h"


@@ -90,7 +91,7 @@ static int stdout_stream_log(StdoutStream *s, const char *p) {
         priority = s->priority;
 
         if (s->level_prefix)
         priority = s->priority;
 
         if (s->level_prefix)

-                syslog_parse_priority((char**) &p, &priority);
+                syslog_parse_priority((char**) &p, &priority, false);

 
         if (s->forward_to_syslog || s->server->forward_to_syslog)
                 server_forward_syslog(s->server, syslog_fixup_facility(priority), s->identifier, p, &s->ucred, NULL);
 
         if (s->forward_to_syslog || s->server->forward_to_syslog)
                 server_forward_syslog(s->server, syslog_fixup_facility(priority), s->identifier, p, &s->ucred, NULL);


@@ -127,7 +128,7 @@ static int stdout_stream_log(StdoutStream *s, const char *p) {
         }
 #endif
 
         }
 #endif
 

-        server_dispatch_message(s->server, iovec, n, ELEMENTSOF(iovec), &s->ucred, NULL, label, label_len, s->unit_id, priority);
+        server_dispatch_message(s->server, iovec, n, ELEMENTSOF(iovec), &s->ucred, NULL, label, label_len, s->unit_id, priority, 0);

 
         free(message);
         free(syslog_priority);
 
         free(message);
         free(syslog_priority);


@@ -323,7 +324,7 @@ void stdout_stream_free(StdoutStream *s) {
         if (s->server) {
                 assert(s->server->n_stdout_streams > 0);
                 s->server->n_stdout_streams --;
         if (s->server) {
                 assert(s->server->n_stdout_streams > 0);
                 s->server->n_stdout_streams --;

-                LIST_REMOVE(StdoutStream, stdout_stream, s->server->stdout_streams, s);
+                LIST_REMOVE(stdout_stream, s->server->stdout_streams, s);

         }
 
         if (s->fd >= 0) {
         }
 
         if (s->fd >= 0) {


@@ -339,6 +340,7 @@ void stdout_stream_free(StdoutStream *s) {
 #endif
 
         free(s->identifier);
 #endif
 
         free(s->identifier);

+        free(s->unit_id);

         free(s);
 }
 
         free(s);
 }
 


@@ -381,8 +383,10 @@ int stdout_stream_new(Server *s) {
         }
 
 #ifdef HAVE_SELINUX
         }
 
 #ifdef HAVE_SELINUX

-        if (getpeercon(fd, &stream->security_context) < 0 && errno != ENOPROTOOPT)
-                log_error("Failed to determine peer security context: %m");
+        if (use_selinux()) {
+                if (getpeercon(fd, &stream->security_context) < 0 && errno != ENOPROTOOPT)
+                        log_error("Failed to determine peer security context: %m");
+        }

 #endif
 
         if (shutdown(fd, SHUT_WR) < 0) {
 #endif
 
         if (shutdown(fd, SHUT_WR) < 0) {


@@ -401,7 +405,7 @@ int stdout_stream_new(Server *s) {
         }
 
         stream->server = s;
         }
 
         stream->server = s;

-        LIST_PREPEND(StdoutStream, stdout_stream, s->stdout_streams, stream);
+        LIST_PREPEND(stdout_stream, s->stdout_streams, stream);

         s->n_stdout_streams ++;
 
         return 0;
         s->n_stdout_streams ++;
 
         return 0;


@@ -440,7 +444,7 @@ int server_open_stdout_socket(Server *s) {
                 chmod(sa.un.sun_path, 0666);
 
                 if (listen(s->stdout_fd, SOMAXCONN) < 0) {
                 chmod(sa.un.sun_path, 0666);
 
                 if (listen(s->stdout_fd, SOMAXCONN) < 0) {

-                        log_error("liste() failed: %m");
+                        log_error("listen() failed: %m");

                         return -errno;
                 }
         } else
                         return -errno;
                 }
         } else