Code

sensors plugin: Avoid assertion in ignorelist_match () when not configured.
authorSebastian Harl <sh@tokkee.org>
Sat, 23 Dec 2006 13:01:14 +0000 (14:01 +0100)
committerFlorian Forster <octo@huhu.verplant.org>
Tue, 9 Jan 2007 13:54:31 +0000 (14:54 +0100)
If the sensors plugin is not configured (i.e. default settings are used) the
ignorelist (sensors_list) is never created causing the assertion in
ignorelist_match () to fail. This fixes it.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
src/sensors.c

index d15569c03a95d58ad361fa7c239ba4faebdd8dba..66cd5b226dea80f8fc22366af2c2a769c7fd59b1 100644 (file)
@@ -393,7 +393,8 @@ static void sensors_shutdown (void)
        sensors_free_features ();
 #endif /* if SENSORS_HAVE_READ */
 
-       ignorelist_free (sensor_list);
+       if (NULL != sensor_list)
+               ignorelist_free (sensor_list);
 }
 
 static void sensors_voltage_write (char *host, char *inst, char *val)
@@ -402,7 +403,7 @@ static void sensors_voltage_write (char *host, char *inst, char *val)
        int status;
 
        /* skip ignored in our config */
-       if (ignorelist_match (sensor_list, inst))
+       if ((NULL != sensor_list) && ignorelist_match (sensor_list, inst))
                return;
 
        /* extended sensor naming */
@@ -423,7 +424,7 @@ static void sensors_write (char *host, char *inst, char *val)
        int status;
 
        /* skip ignored in our config */
-       if (ignorelist_match (sensor_list, inst))
+       if ((NULL != sensor_list) && ignorelist_match (sensor_list, inst))
                return;
 
        /* extended sensor naming */
@@ -450,7 +451,7 @@ static void sensors_submit (const char *feat_name,
                return;
 
        /* skip ignored in our config */
-       if (ignorelist_match (sensor_list, inst))
+       if ((NULL != sensor_list) && ignorelist_match (sensor_list, inst))
                return;
 
        if (snprintf (buf, BUFSIZE, "%u:%.3f", (unsigned int) curtime,