socket: nullify pointers after free

A socket shouldn't be used after socket_done() returns, but follow the
general guideline here and avoid dangling pointers anyway.

diff --git a/src/core/socket.c b/src/core/socket.c
index fb7632d..d6b0c96 100644 (file)
--- a/src/core/socket.c
+++ b/src/core/socket.c
@@ -156,14 +156,14 @@ static void socket_done(Unit *u) {
         s->tcp_congestion = mfree(s->tcp_congestion);
         s->bind_to_device = mfree(s->bind_to_device);
 
-        free(s->smack);
-        free(s->smack_ip_in);
-        free(s->smack_ip_out);
+        s->smack = mfree(s->smack);
+        s->smack_ip_in = mfree(s->smack_ip_in);
+        s->smack_ip_out = mfree(s->smack_ip_out);
 
         strv_free(s->symlinks);
 
-        free(s->user);
-        free(s->group);
+        s->user = mfree(s->user);
+        s->group = mfree(s->group);
 
         s->fdname = mfree(s->fdname);