src/rrd_client.c: Fix a potential segmentation fault in "get_path".

The "get_path" function is called at least from "rrdc_update" without
checking for a connection first. In that case the "sd_path" pointer may be
NULL, so dereferencing it without checking for NULL is not good behavior.

This patch checks all arguments of the "get_path" function and returns an
error if any pointer is NULL. -- Florian Forster

git-svn-id: svn://svn.oetiker.ch/rrdtool/trunk@2094 a5681a0c-68f1-0310-ab6d-d61299d08faa

diff --git a/program/src/rrd_client.c b/program/src/rrd_client.c
index d2922813e099083dd58d3788fd5b1f8608c9c7ff..0d2128387a8ed7633ca3125fb5a9b2d0d571f29f 100644 (file)
--- a/program/src/rrd_client.c
+++ b/program/src/rrd_client.c
@@ -73,6 +73,9 @@ static const char *get_path (const char *path, char *resolved_path) /* {{{ */
   const char *ret = path;
   int is_unix = 0;
 
+  if ((path == NULL) || (resolved_path == NULL) || (sd_path == NULL))
+    return (NULL);
+
   if ((*sd_path == '/')
       || (strncmp ("unix:", sd_path, strlen ("unix:")) == 0))
     is_unix = 1;