From 18d10d885461eaf44fba87ed57a347b565e94f55 Mon Sep 17 00:00:00 2001 From: Donald Sharp Date: Mon, 17 Dec 2018 18:31:09 -0500 Subject: [PATCH] zebra: The mask and sin_mask are a bit redundant for kernel_rtm The test we were using to ensure that a mask was sent in is a bit redundant, let's just always send it in. Signed-off-by: Donald Sharp --- zebra/rt_socket.c | 31 +++++++++---------------------- 1 file changed, 9 insertions(+), 22 deletions(-) diff --git a/zebra/rt_socket.c b/zebra/rt_socket.c index 0336d79ee..539370a49 100644 --- a/zebra/rt_socket.c +++ b/zebra/rt_socket.c @@ -118,7 +118,6 @@ static int kernel_rtm(int cmd, const struct prefix *p, const struct nexthop_group *ng, uint32_t metric) { - union sockunion *mask = NULL; union sockunion sin_dest, sin_mask, sin_gate; #ifdef __OpenBSD__ struct sockaddr_mpls smpls; @@ -237,32 +236,20 @@ static int kernel_rtm(int cmd, const struct prefix *p, switch (p->family) { case AF_INET: - if (gate && p->prefixlen == 32) - mask = NULL; - else { - masklen2ip(p->prefixlen, - &sin_mask.sin.sin_addr); - sin_mask.sin.sin_family = AF_INET; + masklen2ip(p->prefixlen, &sin_mask.sin.sin_addr); + sin_mask.sin.sin_family = AF_INET; #ifdef HAVE_STRUCT_SOCKADDR_IN_SIN_LEN - sin_mask.sin.sin_len = sin_masklen( - sin_mask.sin.sin_addr); + sin_mask.sin.sin_len = sin_masklen( + sin_mask.sin.sin_addr); #endif /* HAVE_STRUCT_SOCKADDR_IN_SIN_LEN */ - mask = &sin_mask; - } break; case AF_INET6: - if (gate && p->prefixlen == 128) - mask = NULL; - else { - masklen2ip6(p->prefixlen, - &sin_mask.sin6.sin6_addr); - sin_mask.sin6.sin6_family = AF_INET6; + masklen2ip6(p->prefixlen, &sin_mask.sin6.sin6_addr); + sin_mask.sin6.sin6_family = AF_INET6; #ifdef SIN6_LEN - sin_mask.sin6.sin6_len = sin6_masklen( - sin_mask.sin6.sin6_addr); + sin_mask.sin6.sin6_len = sin6_masklen( + sin_mask.sin6.sin6_addr); #endif /* SIN6_LEN */ - mask = &sin_mask; - } break; } @@ -272,7 +259,7 @@ static int kernel_rtm(int cmd, const struct prefix *p, continue; smplsp = (union sockunion *)&smpls; #endif - error = rtm_write(cmd, &sin_dest, mask, + error = rtm_write(cmd, &sin_dest, &sin_mask, gate ? &sin_gate : NULL, smplsp, ifindex, bh_type, metric); -- 2.39.2