conn_sock: fix double free

Signed-off-by: Peter Hunt <pehunt@redhat.com>
containers · Jan 20, 2021 · d9bd8f8 · d9bd8f8
1 parent 3af2776
commit d9bd8f8
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/conn_sock.c b/src/conn_sock.c
@@ -170,9 +170,10 @@ static char *bind_unix_socket(char *socket_relative_name, int sock_type, mode_t
 
 
 	/*
-	 * we use the fullpath for operations that aren't as limited in length as socket_addr.sun_path
+	 * We use the fullpath for operations that aren't as limited in length as socket_addr.sun_path
+	 * Cleanup of this variable is up to the caller
 	 */
-	_cleanup_free_ char *sock_fullpath = g_build_filename(parent_dir, socket_relative_name, NULL);
+	char *sock_fullpath = g_build_filename(parent_dir, socket_relative_name, NULL);
 
 	/*
 	 * We make the socket non-blocking to avoid a race where client aborts connection