Code

src/rrd_client.c: Fix a potential segmentation fault in "get_path".
authoroetiker <oetiker@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Wed, 30 Jun 2010 12:48:47 +0000 (12:48 +0000)
committeroetiker <oetiker@a5681a0c-68f1-0310-ab6d-d61299d08faa>
Wed, 30 Jun 2010 12:48:47 +0000 (12:48 +0000)
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

program/src/rrd_client.c

index d2922813e099083dd58d3788fd5b1f8608c9c7ff..0d2128387a8ed7633ca3125fb5a9b2d0d571f29f 100644 (file)
@@ -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;