Subject: [PATCH 1/5] Null check before dereference

[PATCH 1/5] Null check before dereference

From: Gregor Jasny <gjasny_at_googlemail.com>
Date: Sun, 28 Sep 2014 22:19:23 +0200

Fix Coverity error CID 56880

Signed-off-by: Gregor Jasny <gjasny_at_googlemail.com>

---
 ares_options.c | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/ares_options.c b/ares_options.c
index 76d82df..cf88433 100644
--- a/ares_options.c
+++ b/ares_options.c
@@ -158,6 +158,9 @@ int ares_set_servers_csv(ares_channel channel,
      return ARES_SUCCESS; /* blank all servers */
 
   csv = malloc(i + 2);
+  if (!csv)
+    return ARES_ENOMEM;
+
   strcpy(csv, _csv);
   if (csv[i-1] != ',') { /* make parsing easier by ensuring ending ',' */
     csv[i] = ',';
-- 
1.9.3 (Apple Git-50)
Received on 2014-09-28