From: Ciara Loftus <ciara.loftus@intel.com>
Date: Tue, 29 May 2018 13:46:31 +0000 (+0100)
Subject: utils_ovs: Avoid potential access of freed memory
X-Git-Url: https://git.verplant.org/?a=commitdiff_plain;h=190de6276c305b7d6b83d7911a44b4057e2ebd8d;p=collectd.git

utils_ovs: Avoid potential access of freed memory

Fixes #2800
Signed-off-by: Ciara Loftus <ciara.loftus@intel.com>
---

diff --git a/src/utils_ovs.c b/src/utils_ovs.c
index 3b7236ca..4ca86aeb 100644
--- a/src/utils_ovs.c
+++ b/src/utils_ovs.c
@@ -1051,6 +1051,8 @@ ovs_db_t *ovs_db_init(const char *node, const char *service,
     ret = ovs_db_destroy(pdb);
     if (ret > 0)
       goto failure;
+    else
+      return NULL;
   }
 
   /* init polling thread */
@@ -1059,6 +1061,8 @@ ovs_db_t *ovs_db_init(const char *node, const char *service,
     if (ret > 0) {
       ovs_db_event_thread_data_destroy(pdb);
       goto failure;
+    } else {
+      return NULL;
     }
   }
   return pdb;