-
assigned issue to
Tildeslash
- edited description
small ICMP echo reply from different address than being checked causes issue
Issue #488
resolved
this is seen for monit 5.4 and also monit 5.19 In trying to use the ICMP type echo / ICMP ping functionality. We are using the ping functionality to check accessibility of another machine. If there happens to be another set of machines on the network that are using an ICMP ping of a size smaller than what monit uses, monit sees the ping reply and complains about the reply being too small. For example:
[EDT Jun 27 13:58:51] error : ICMP echo response for 169.254.99.1 1/1 failed -- received 40 bytes, expected at least 48 bytes
The response in this case is meant for a different address, yet it causes a failure for my check condition. The monit code should not treat the small reply as being a failure when it is not for the desired address. The address check should be performed before the size check.
Comments (9)
-
repo owner
-
repo owner
The data which broke the response were not ICMP ping, but probably some other message type ... the minimum length expected by monit is just "sizeof(struct ip) + sizeof(struct icmp)" for ICMPv4 or sizeof(struct icmp6_hdr) for ICMPv6.
Will fix.
-
repo owner
- changed status to resolved
Fixed: Issue
#488: Ping test may fail.→ <<cset f5692a4b5654>>
-
reporter
any idea when a new version will be released with this fix?
-
repo owner
it'll be part of next maintenance release, ca. 1-2 months
You can download the development repository snapshot using https://bitbucket.org/tildeslash/monit/get/master.tar.gz
To compile:
./bootstrap ./configure make -
reporter
is there a date yet for when monit 5.21 will be released? we are past the 1-2 month maintenance release period and are looking to plan when we will have it available
-
repo owner
It should be released probably during next week (tentative). We work on disk IO monitoring, which we'd like to include in the next release.
-
reporter
any update on when monit 5.21 will be released? i release the last comment listed the date as tentative, but we are now another month out.
thanks!
-
repo owner
The disk IO monitoring is finished, we're now in the testing phase + walking through bitbucket issues (small fixes in progress). If you need to use the latest code base, you can always get the development snapshot from bitbucket:
wget https://bitbucket.org/tildeslash/monit/get/master.tar.gz tar -xzf master.tar.gz cd tildeslash* ./bootstrap ./configure make - Log in to comment
