diff options
author | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-06-14 01:36:54 +0200 |
---|---|---|
committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2019-06-20 01:07:12 +0200 |
commit | 11884868d4b091a7e82e80258d676122239ad6b8 (patch) | |
tree | f9810a5bc65c7d64fefa2f3259b2e0ab9ad4d3e5 /pimd/pim_ifchannel.c | |
parent | lib: Do not blindly set the _read and _write pointers to NULL (diff) | |
download | frr-11884868d4b091a7e82e80258d676122239ad6b8.tar.xz frr-11884868d4b091a7e82e80258d676122239ad6b8.zip |
pimd: ALLOC functions cannot fail.
There is no need to check for ALLOC function failures
in the code base. If we cannot get more memory we
assert.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_ifchannel.c')
-rw-r--r-- | pimd/pim_ifchannel.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/pimd/pim_ifchannel.c b/pimd/pim_ifchannel.c index 3fb3da8f4..1460c8bb6 100644 --- a/pimd/pim_ifchannel.c +++ b/pimd/pim_ifchannel.c @@ -791,8 +791,6 @@ void pim_ifchannel_join_add(struct interface *ifp, struct in_addr neigh_addr, ch = pim_ifchannel_add(ifp, sg, source_flags, PIM_UPSTREAM_FLAG_MASK_SRC_PIM); - if (!ch) - return; /* RFC 4601: 4.6.1. (S,G) Assert Message State Machine @@ -953,8 +951,6 @@ void pim_ifchannel_prune(struct interface *ifp, struct in_addr upstream, ch = pim_ifchannel_add(ifp, sg, source_flags, PIM_UPSTREAM_FLAG_MASK_SRC_PIM); - if (!ch) - return; pim_ifp = ifp->info; @@ -1082,13 +1078,6 @@ int pim_ifchannel_local_membership_add(struct interface *ifp, } ch = pim_ifchannel_add(ifp, sg, 0, PIM_UPSTREAM_FLAG_MASK_SRC_IGMP); - if (!ch) { - if (PIM_DEBUG_EVENTS) - zlog_debug("%s:%s Unable to add ifchannel", - __PRETTY_FUNCTION__, - pim_str_sg_dump(sg)); - return 0; - } ifmembership_set(ch, PIM_IFMEMBERSHIP_INCLUDE); |