upstream/monitoring-plugins
1
0
Fork 0
mirror of https://github.com/monitoring-plugins/monitoring-plugins.git synced 2026-04-21 14:19:30 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

check_host: Allocate a large-enough buffer for the host table.

Browse source 
When specifying a host-name on the command line, each of its IPs is added to
the host table (and each one is pinged). So, the buffer has to be large enough
to hold all of the respective host objects. (argc - 1) only fits hosts with a
single IP.

Thanks to Max Kosmach <max@tcen.ru> for reporting this in Debian bug #623702.
This commit is contained in:
Sebastian Harl 2011-04-28 09:18:21 +02:00 committed by Thomas Guyot-Sionnest
parent 1bc7a4a198
commit 5ebe25fc24
3 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
NEWS
View file

@ -14,6 +14,7 @@ This file documents the major additions and syntax changes between releases.
Make check_snmp work more like v1.4.14 with regard to using special values (Timeticks, STRING) as numeric thresholds
Fix check_ldap overriding the port when --ssl was specified after -p
Fix check_procs where regex input of '|' would get displayed in output - now replaced with ','
Fix segfault in check_host when hostname returns multiple IP addresses (Sebastian Harl)
1.4.15 27th July 2010
ENHANCEMENTS

1
THANKS.in
View file

@ -268,3 +268,4 @@ Brian Landers
Ryan Kelly
Stéphane Urbanovski
Marco Beck
Sebastian Harl

2
plugins-root/check_icmp.c
View file

@ -621,7 +621,7 @@ main(int argc, char **argv)
}
host = list;
table = malloc(sizeof(struct rta_host **) * (argc - 1));
table = malloc(sizeof(struct rta_host **) * targets);
i = 0;
while(host) {
host->id = i*packets;