From: Alex Wang <alexw@nicira.com>
Date: Tue, 18 Aug 2015 06:12:55 +0000 (-0700)
Subject: ovn-controller-vtep: Make 'ovnsb_remote' and 'vtep_remote' free-able.
X-Git-Tag: v2.5.0~728
X-Git-Url: http://git.cascardo.eti.br/?a=commitdiff_plain;h=59e5e6a1a94dde75137776dc442627ade42d1542;hp=ae4b9136e5003dd1637585a578c522673161da7d;p=cascardo%2Fovs.git

ovn-controller-vtep: Make 'ovnsb_remote' and 'vtep_remote' free-able.

Found by inspection.

Signed-off-by: Alex Wang <alexw@nicira.com>
Acked-by: Russell Bryant <rbryant@redhat.com>
---

diff --git a/ovn/controller-vtep/ovn-controller-vtep.c b/ovn/controller-vtep/ovn-controller-vtep.c
index ff2531914..7e98f6901 100644
--- a/ovn/controller-vtep/ovn-controller-vtep.c
+++ b/ovn/controller-vtep/ovn-controller-vtep.c
@@ -48,6 +48,7 @@ OVS_NO_RETURN static void usage(void);
 
 static char *vtep_remote;
 static char *ovnsb_remote;
+static char *default_db_;
 
 int
 main(int argc, char *argv[])
@@ -142,19 +143,19 @@ main(int argc, char *argv[])
 
     free(ovnsb_remote);
     free(vtep_remote);
+    free(default_db_);
     service_stop();
 
     exit(retval);
 }
 
-static char *
+static const char *
 default_db(void)
 {
-    static char *def;
-    if (!def) {
-        def = xasprintf("unix:%s/db.sock", ovs_rundir());
+    if (!default_db_) {
+        default_db_ = xasprintf("unix:%s/db.sock", ovs_rundir());
     }
-    return def;
+    return default_db_;
 }
 
 static void
@@ -189,11 +190,11 @@ parse_options(int argc, char *argv[])
 
         switch (c) {
         case 'd':
-            ovnsb_remote = optarg;
+            ovnsb_remote = xstrdup(optarg);
             break;
 
         case 'D':
-            vtep_remote = optarg;
+            vtep_remote = xstrdup(optarg);
             break;
 
         case 'h':
@@ -224,11 +225,11 @@ parse_options(int argc, char *argv[])
     argv += optind;
 
     if (!ovnsb_remote) {
-        ovnsb_remote = default_db();
+        ovnsb_remote = xstrdup(default_db());
     }
 
     if (!vtep_remote) {
-        vtep_remote = default_db();
+        vtep_remote = xstrdup(default_db());
     }
 }