diff options
| author |  Thomas Guyot-Sionnest <dermoth@aei.ca>
| 2010-11-30 22:48:43 -0500 |
|---|---|---|
| committer | Thomas Guyot-Sionnest <dermoth@aei.ca>
| 2010-11-30 22:48:43 -0500 |
| commit | 5a2814a21bff07b87a7589ef19b63c9eecb8be9f (patch) | |
| tree | e112f9645005cb6fb84b96c11640dd613fe14e0c /plugins/check_snmp.c | |
| parent | 811684ffe394d050158c2a98d5be211d5ded3c88 (diff) | |
| download | monitoring-plugins-5a2814a21bff07b87a7589ef19b63c9eecb8be9f.tar.gz | |
Revert "check_snmp now considers strings returned by SNMP that contain just"
This reverts commit 896962a1ad1b7d7c75d42c565b06cc799feb0a7c.
Conflicts:
NEWS
plugins/tests/check_snmp.t
Notes:
Reverting because I rebased a patch that was doing the same thing, plus
fixing more related regressions, and both didn't work together.
I kept the tests intact except for one that wouldn't pass on 1.4.14
either
Diffstat (limited to 'plugins/check_snmp.c')
| -rw-r--r-- | plugins/check_snmp.c | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/plugins/check_snmp.c b/plugins/check_snmp.c index cb7fb7a0..d79da8cf 100644 --- a/plugins/check_snmp.c +++ b/plugins/check_snmp.c @@ -160,7 +160,6 @@ main (int argc, char **argv) char *outbuff; char *ptr = NULL; char *show = NULL; - char *endptr = NULL; char *th_warn=NULL; char *th_crit=NULL; char type[8] = ""; @@ -390,19 +389,6 @@ main (int argc, char **argv) } } - /* Allow numeric conversion if whole string is a number. Make concession for strings with " at beginning or end */ - /* This duplicates the conversion a bit later, but is cleaner to separate out the checking against the conversion */ - ptr = show; - if (*ptr == '"') - ptr++; - if (*ptr != '\0' ) { - strtod( ptr, &endptr ); - if (*endptr == '"') - endptr++; - if (*endptr == '\0') - is_numeric=1; - } - } else if (strstr (response, "Timeticks: ")) { show = strstr (response, "Timeticks: "); |