unixctl: Fix bad log message on error path.
authorBen Pfaff <blp@nicira.com>
Fri, 23 Mar 2012 20:42:12 +0000 (13:42 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 23 Mar 2012 20:56:22 +0000 (13:56 -0700)
'path' is usually not the right path (often it's NULL).

Introduced in commit bde9f75de (unixctl: New JSON RPC back-end.).

Signed-off-by: Ben Pfaff <blp@nicira.com>
lib/unixctl.c

index 054ce49..d8974c9 100644 (file)
@@ -222,12 +222,9 @@ unixctl_server_create(const char *path, struct unixctl_server **serverp)
     }
 
     error = pstream_open(punix_path, &listener);
-    free(punix_path);
-    punix_path = NULL;
-
     if (error) {
-        ovs_error(error, "could not initialize control socket %s", path);
-        return error;
+        ovs_error(error, "could not initialize control socket %s", punix_path);
+        goto exit;
     }
 
     unixctl_command_register("help", "", 0, 0, unixctl_help, NULL);
@@ -237,7 +234,10 @@ unixctl_server_create(const char *path, struct unixctl_server **serverp)
     server->listener = listener;
     list_init(&server->conns);
     *serverp = server;
-    return 0;
+
+exit:
+    free(punix_path);
+    return error;
 }
 
 static void