From: paco Date: Tue, 26 Jun 2018 15:31:00 +0000 (+0200) Subject: pimd: return check (Coverity 1465490) X-Git-Tag: frr-6.1-dev~241^2 X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=d1b21b9616e3fda279e9bc28f16764cbab9859ec;p=matthieu%2Ffrr.git pimd: return check (Coverity 1465490) Unless someone intentionally changes MCAST_ALL_ROUTERS ("224.0.0.2") with a wrong IP, this should never fail, so the fix is using "(void)" at the left of the function call, as an explicit way of indicating we discard the return value on purpose. Signed-off-by: F. Aragon --- diff --git a/pimd/pim_igmp_mtrace.c b/pimd/pim_igmp_mtrace.c index 673e2ca5b8..95d0278a34 100644 --- a/pimd/pim_igmp_mtrace.c +++ b/pimd/pim_igmp_mtrace.c @@ -817,7 +817,7 @@ int igmp_mtrace_recv_qry_req(struct igmp_sock *igmp, struct ip *ip_hdr, * Previous-hop router not known, * packet is sent to an appropriate multicast address */ - inet_aton(MCAST_ALL_ROUTERS, &nh_addr); + (void)inet_aton(MCAST_ALL_ROUTERS, &nh_addr); } /* 6.2.2 8. If this router is the Rendez-vous Point */