diff options
| author | Aksel Sjögren <asjogren@itrsgroup.com> | 2021-11-15 14:29:51 +0100 |
|---|---|---|
| committer | Sven Nierlein <sven@nierlein.org> | 2021-11-15 17:55:49 +0100 |
| commit | 3f9b22ab16b5a9b74d62d7fff9b6bf3a51ed93f7 (patch) | |
| tree | b6cdc78a7c2197b5b1fb01a0910d5110dbbbcda3 | |
| parent | bc6effd2b53e58bb8aec6fc4b4465a3927b518d2 (diff) | |
| download | monitoring-plugins-3f9b22ab16b5a9b74d62d7fff9b6bf3a51ed93f7.tar.gz | |
check_icmp: Fix pkt perfdata in check_host mode
Add missing "warn" threshold field in "pkt" perfdata output.
Perfdata should be interpreted as;
'label'=value[UOM];[warn];[crit];[min];[max]
With one field missing, the hardcoded min value '0' ended up in the
"crit" field, making applications interpreting the perfdata thining that
critical threshold is always exceeded.
Signed-off-by: Aksel Sjögren <asjogren@itrsgroup.com>
| -rw-r--r-- | plugins-root/check_icmp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/plugins-root/check_icmp.c b/plugins-root/check_icmp.c index 31eb4c65..519b93c6 100644 --- a/plugins-root/check_icmp.c +++ b/plugins-root/check_icmp.c | |||
| @@ -921,7 +921,7 @@ wait_for_reply(int sock, u_int t) | |||
| 921 | /* if we're in hostcheck mode, exit with limited printouts */ | 921 | /* if we're in hostcheck mode, exit with limited printouts */ |
| 922 | if(mode == MODE_HOSTCHECK) { | 922 | if(mode == MODE_HOSTCHECK) { |
| 923 | printf("OK - %s responds to ICMP. Packet %u, rta %0.3fms|" | 923 | printf("OK - %s responds to ICMP. Packet %u, rta %0.3fms|" |
| 924 | "pkt=%u;;0;%u rta=%0.3f;%0.3f;%0.3f;;\n", | 924 | "pkt=%u;;;0;%u rta=%0.3f;%0.3f;%0.3f;;\n", |
| 925 | host->name, icmp_recv, (float)tdiff / 1000, | 925 | host->name, icmp_recv, (float)tdiff / 1000, |
| 926 | icmp_recv, packets, (float)tdiff / 1000, | 926 | icmp_recv, packets, (float)tdiff / 1000, |
| 927 | (float)warn.rta / 1000, (float)crit.rta / 1000); | 927 | (float)warn.rta / 1000, (float)crit.rta / 1000); |
