Code

snmp plugin: Fix off-by-one array access.
authorFlorian Forster <octo@collectd.org>
Wed, 17 Jun 2015 17:12:45 +0000 (19:12 +0200)
committerFlorian Forster <octo@collectd.org>
Wed, 17 Jun 2015 20:09:14 +0000 (22:09 +0200)
src/snmp.c

index 7d340d18c2398a8a26fb845d7b0e03352b53f903..362f175cd33fba30e41c12c0555534a46ef9befd 100644 (file)
@@ -1341,7 +1341,7 @@ static int csnmp_read_table (host_definition_t *host, data_definition_t *data)
     for (vb = res->variables, i = 0; (vb != NULL); vb = vb->next_variable, i++)
     {
       /* Calculate value index from todo list */
-      while (!oid_list_todo[i] && (i < oid_list_len))
+      while ((i < oid_list_len) && !oid_list_todo[i])
         i++;
 
       /* An instance is configured and the res variable we process is the