diff options
author | Justin Pettit <jpettit@nicira.com> | 2009-11-16 16:36:21 -0800 |
---|---|---|
committer | Justin Pettit <jpettit@nicira.com> | 2009-11-16 18:11:39 -0800 |
commit | fb344b867b96a6d6b9ec32e277cae6db68f2f236 (patch) | |
tree | 276384704d8aa3c741f872ee42f64d84e1d1b8e9 /utilities/ovs-appctl.c | |
parent | 4836f9f2bfa5dc0531bd1d828aae96ccd6b415cb (diff) | |
download | openvswitch-fb344b867b96a6d6b9ec32e277cae6db68f2f236.tar.gz |
ovs-appctl: Fix shadow variable that could cause segfault
The variable "socket_name" contains the name of the unix domain socket
to be used for communicating with the OVS process. If the target does
not begin with a "/", the socket name is determined based on a pidfile.
A shadow copy of "socket_name" was kept in the block that looks at the
pidfile, which would cause the function-level one to not be set. This
removes that shadow copy.
Diffstat (limited to 'utilities/ovs-appctl.c')
-rw-r--r-- | utilities/ovs-appctl.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/utilities/ovs-appctl.c b/utilities/ovs-appctl.c index 060e8e286..1302bf2e0 100644 --- a/utilities/ovs-appctl.c +++ b/utilities/ovs-appctl.c @@ -170,7 +170,6 @@ connect_to_target(const char *target) if (target[0] != '/') { char *pidfile_name; - char *socket_name; pid_t pid; pidfile_name = xasprintf("%s/%s.pid", ovs_rundir, target); |