Andrew,
I think pindg is rather broken:
strace -f /usr/lib64/heartbeat/pingd -a pingdnet2 -d 5s -i 1 -n 2 -h
10.0.1.16
(which is in fact localhost)
In an endless loop:
sendmsg(4, {msg_name(16)={sa_family=AF_INET, sin_port=htons(0),
sin_addr=inet_addr("10.0.1.16")}, msg_iov(1)=[{"\10\0\324\fj\276\0\2pingd-
v4\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0"..., 39}], msg_controllen=0, msg_flags=0},
0) = 39
recvmsg(4, {msg_name(16)={sa_family=AF_INET, sin_port=htons(0),
sin_addr=inet_addr("192.168.2.16")},
msg_iov(1)=[{"e\320\1\22y\35\...@\1\232\215\300\250\2\20\300\250\2\20\3\3b;
\0\0\0\0E\20\0\366"..., 324}], msg_controllen=0, msg_flags=0}, 0) = 274
recvmsg(4, 0x7fff498799d0, MSG_ERRQUEUE|MSG_DONTWAIT) = -1 EAGAIN (Resource
temporarily unavailable)
write(2, "No error message: -1: Resource t"..., 60No error message: -1:
Resource temporarily unavailable (11)
My proposal is to rip out all network code out of pingd and to add slightly
modified files from 'iputils'. As ping files from iputils are under a BSD
license, those also really should go into different files. I also think there
already is an existing licence violation, as pingd.c includes a function from
iputils ping. While the function is marked accordingly, it still does not
include the original license statement, which is IMHO a clear license
violation.
I could probably do that quicky, but don't want to do something that is not
accepted upstream. Please let me know your thoughts.
Thanks,
Bernd
--
Bernd Schubert
DataDirect Networks
_______________________________________________
Pacemaker mailing list: Pacemaker@oss.clusterlabs.org
http://oss.clusterlabs.org/mailman/listinfo/pacemaker
Project Home: http://www.clusterlabs.org
Getting started: http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf
Bugs: http://developerbugs.linux-foundation.org/enter_bug.cgi?product=Pacemaker
