summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndreas Ericsson <ae@op5.se>2012-11-02 14:52:34 +0100
committerThomas Guyot-Sionnest <dermoth@aei.ca>2012-11-02 22:25:37 -0400
commitc537d71a3af67d0df4e1e015f6b9dcbb505f08d2 (patch)
tree5710a59c7477444fd29f6dee37bd9230709419da
parent364ce21b1048d1125b8d3fd6744661ab581d08c7 (diff)
downloadmonitoring-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>
-rw-r--r--plugins/check_snmp.c4
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) {