diff options
author | Andreas Ericsson <ae@op5.se> | 2012-11-02 14:52:34 +0100 |
---|---|---|
committer | Thomas Guyot-Sionnest <dermoth@aei.ca> | 2012-11-02 22:25:37 -0400 |
commit | c537d71a3af67d0df4e1e015f6b9dcbb505f08d2 (patch) | |
tree | 5710a59c7477444fd29f6dee37bd9230709419da /plugins | |
parent | 364ce21b1048d1125b8d3fd6744661ab581d08c7 (diff) | |
download | monitoring-plugins-c537d71a3af67d0df4e1e015f6b9dcbb505f08d2.tar.gz |
plugins/check_snmp: Show snmp error when we fail to parse its output
Dying without a hint of what went wrong is just plain annoying. With
this patch we at least get a hint.
While at it, we fix the string offset so the output we're interested
in doesn't keep the equal sign that snmpget prints.
Signed-off-by: Andreas Ericsson <ae@op5.se>
Diffstat (limited to 'plugins')
-rw-r--r-- | plugins/check_snmp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/plugins/check_snmp.c b/plugins/check_snmp.c index 0ddfb942..8a8ee180 100644 --- a/plugins/check_snmp.c +++ b/plugins/check_snmp.c | |||
@@ -419,7 +419,7 @@ main (int argc, char **argv) | |||
419 | show = strstr (response, "Timeticks: "); | 419 | show = strstr (response, "Timeticks: "); |
420 | } | 420 | } |
421 | else | 421 | else |
422 | show = response; | 422 | show = response + 3; |
423 | 423 | ||
424 | iresult = STATE_DEPENDENT; | 424 | iresult = STATE_DEPENDENT; |
425 | 425 | ||
@@ -428,7 +428,7 @@ main (int argc, char **argv) | |||
428 | if (thlds[i]->warning || thlds[i]->critical || calculate_rate) { | 428 | if (thlds[i]->warning || thlds[i]->critical || calculate_rate) { |
429 | ptr = strpbrk (show, "0123456789"); | 429 | ptr = strpbrk (show, "0123456789"); |
430 | if (ptr == NULL) | 430 | if (ptr == NULL) |
431 | die (STATE_UNKNOWN,_("No valid data returned")); | 431 | die (STATE_UNKNOWN,_("No valid data returned (%s)\n"), show); |
432 | response_value[i] = strtod (ptr, NULL); | 432 | response_value[i] = strtod (ptr, NULL); |
433 | 433 | ||
434 | if(calculate_rate) { | 434 | if(calculate_rate) { |