Hide Forgot
Created attachment 517169 [details] fix for "1" These errors was revealed by coverity difference scan on packages from rhel6.1/rhel6.2 ERROR LIST: 1) tools/findif.c:547: Missing check for return value of inet_pton call in ConvertQuadToint function. 2) tools/sfex_init.c: Missing check for write_lockdata return value. 3) heartbeat/IPv6addr.c:434: bad check for socket(3) return value .. returns -1 when error occurred 4) heartbeat/IPv6addr.c:706: missing check for socket(3) return value 5) heartbeat/IPv6addr.c:704: msg.msg_flags is not initialized before recvmsg 6) tools/findif.c:{279|271}: unused value "cp" 7) tools/send_arp.linux.c:360: Missing break? It would be nice if somebody could look at these defects. Pavel
Created attachment 517170 [details] fix for "2"
Created attachment 517172 [details] fix for "3"
Created attachment 517173 [details] fix for "4"
Created attachment 517175 [details] fix for "5"
Created attachment 517176 [details] fix for "6"
Created attachment 517179 [details] fix for "3" repaired
This request was not resolved in time for the current release. Red Hat invites you to ask your support representative to propose this request, if still desired, for consideration in the next release of Red Hat Enterprise Linux.
This request was erroneously removed from consideration in Red Hat Enterprise Linux 6.4, which is currently under development. This request will be evaluated for inclusion in Red Hat Enterprise Linux 6.4.
First one, probably least straightforward, addressed by pull request [1]. [1] https://github.com/ClusterLabs/resource-agents/pull/143
I think 5) can be safely dropped as msg_flags item of struct msghdr: - is supposed to be set in recvmsg (RECVMSG(2) is clear, RECVMSG(3p) seems to allow for both interpretation) - is not examined upon return from recvmsg
So, currently the separation into pull requests is as follows: 1, 6: [1] 3, 4: [2] 2, 7: [3] (break was not missing, added clarification) 5: omitted completely (see [comment 16]) [2] https://github.com/ClusterLabs/resource-agents/pull/144 [3] https://github.com/ClusterLabs/resource-agents/pull/145 Waiting for upstream's feedback.
The only more significant change worth some kind of validation is related to findif helper. Its test has been added to pull request [1].
moving to 6.5. not relevant for 6.4 as only packemaker agents are affected and they are TP in RHEL6.x.
Need to check if those upstream fixes were downstreamed with [bug 993431] rebase or not.
Jan, have those fixes been included upstream?
> have those fixes been included upstream? Investigating the state of the mentioned 3 PRs, they were merged and initially included in the release 3.9.4. PR#143 even made it to the changelog: [1] https://github.com/ClusterLabs/resource-agents/blob/v3.9.4/ChangeLog#L52