-
Notifications
You must be signed in to change notification settings - Fork 329
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FreeBSD 13.1 check_icmp -4|-6 broken #658
Comments
The problem seems to be that the author is passing the size of sockaddr_storage to sendmsg() in the header rather then the size of the actual sockaddr_in/in6 struct contained within. I think this creates a size mismatch with the msg_name field that FreeBSD 13.1 checks for resulting in the EINVAL return code from the sendmsg() call. This patch will get it going again, though I am not sure if it is complete, the receive side may need some changes too, but it seems to work:
The patch also changes ttl to an int instead of an unsigned char which fixes the setsockopt() call for the ttl in FreeBSD. I have not tested if this breaks Linux at all. |
I tested this PR on Linux Kernel 5.17.9 and I had no issues compiling check_icmp or running a test ping with v4 and v6. |
Thanks for the patch and for testing it. I'll try to get this fix into the next release. |
I ran into a problem after upgrade of Nagios Server OS FreeBSD 13.0 to 13.1
check_icmp -4|-6 seems to be broken:
Please also see #587
Bug/Problem was incoming with IPv6 on earlier versions FreeBSD
The text was updated successfully, but these errors were encountered: