check_snmp now only prints perfdata for non numeric values (Joerg Linge #1867716)

git-svn-id: https://nagiosplug.svn.sourceforge.net/svnroot/nagiosplug/nagiosplug/trunk@2021 f882894a-f735-0410-b71e-b25c423dba1c

1 files changed, 11 insertions, 7 deletions

diff --git a/plugins/check_snmp.c b/plugins/check_snmp.c
index 0acade25..544c5c0b 100644
--- a/plugins/check_snmp.c
+++ b/plugins/check_snmp.c
@@ -228,14 +228,14 @@ main (int argc, char **argv)
 
         strncat(perfstr, "| ", sizeof(perfstr)-strlen(perfstr)-1);
         while (ptr) {
-                char *foo;
+                char *foo, *ptr2;
                 unsigned int copylen;
 
                 foo = strstr (ptr, delimiter);
                 copylen = foo-ptr;
                 if (copylen > sizeof(perfstr)-strlen(perfstr)-1)
                         copylen = sizeof(perfstr)-strlen(perfstr)-1;
-                strncat(perfstr, ptr, copylen);
+                ptr2 = ptr;
                 ptr = foo; 
 
                 if (ptr == NULL)
@@ -368,11 +368,15 @@ main (int argc, char **argv)
 
                 i++;
 
-                strncat(perfstr, "=", sizeof(perfstr)-strlen(perfstr)-1);
-                strncat(perfstr, show, sizeof(perfstr)-strlen(perfstr)-1);
-                if (type)
-                        strncat(perfstr, type, sizeof(perfstr)-strlen(perfstr)-1);
-                strncat(perfstr, " ", sizeof(perfstr)-strlen(perfstr)-1);
+                if (is_numeric(show)) {
+                        strncat(perfstr, ptr2, copylen);
+                        strncat(perfstr, "=", sizeof(perfstr)-strlen(perfstr)-1);
+                        strncat(perfstr, show, sizeof(perfstr)-strlen(perfstr)-1);
+
+                        if (type)
+                                strncat(perfstr, type, sizeof(perfstr)-strlen(perfstr)-1);
+                        strncat(perfstr, " ", sizeof(perfstr)-strlen(perfstr)-1);
+                }
 
         }       /* end while (ptr) */