diff options
| author | paco <paco@voltanet.io> | 2018-06-26 17:31:00 +0200 |
|---|---|---|
| committer | paco <paco@voltanet.io> | 2018-06-26 19:34:04 +0200 |
| commit | d1b21b9616e3fda279e9bc28f16764cbab9859ec (patch) | |
| tree | 1dc78abdf689d4a61acc7c53711f0298ba9d17b8 /pimd | |
| parent | Merge pull request #2549 from pacovn/Coverity_1452317_Explicit_null_dereferenced (diff) | |
| download | frr-d1b21b9616e3fda279e9bc28f16764cbab9859ec.tar.xz frr-d1b21b9616e3fda279e9bc28f16764cbab9859ec.zip | |
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 <paco@voltanet.io>
Diffstat (limited to 'pimd')
| -rw-r--r-- | pimd/pim_igmp_mtrace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pimd/pim_igmp_mtrace.c b/pimd/pim_igmp_mtrace.c index 673e2ca5b..95d0278a3 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 */ |