aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthew Smith <mgsmith@netgate.com>2021-05-25 09:41:05 -0500
committerNeale Ranns <neale@graphiant.com>2021-06-03 14:13:44 +0000
commit8dd0d75b55ab71ecaab4b3c0240a10a5bf0f252b (patch)
tree9c1a0b42efec868a1c3957512380e4c9383dbd28
parentc5e3a41bbeedcc7486f43edfbc85b63a95865773 (diff)
linux-cp: use default adjacency
Type: fix Outbound packets from the host have an adjacency lookup performed so buffer metadata can be filled in and output features can be applied. If no adjacency is found for a packet, it gets dropped. This breaks DHCP and possibly other things since the DHCP reply to a discover request is sent to a destination MAC address that is determined by the contents of the request packet rather than any existing neighbor table entry. If adjacency lookup for outbound packets does not find an entry, use the default adjacency for the interface & address family. Change-Id: Ia53a4df3a5bad2991768cfe4a84c560b879e656f Signed-off-by: Matthew Smith <mgsmith@netgate.com>
-rw-r--r--src/plugins/linux-cp/lcp_node.c37
1 files changed, 16 insertions, 21 deletions
diff --git a/src/plugins/linux-cp/lcp_node.c b/src/plugins/linux-cp/lcp_node.c
index 7f099ffe856..b2ffc5fb63f 100644
--- a/src/plugins/linux-cp/lcp_node.c
+++ b/src/plugins/linux-cp/lcp_node.c
@@ -339,6 +339,7 @@ lcp_xc_inline (vlib_main_t *vm, vlib_node_runtime_t *node, vlib_frame_t *frame,
const lcp_itf_pair_t *lip;
u32 next0, bi0, lipi, ai;
vlib_buffer_t *b0;
+ const ip_adjacency_t *adj;
bi0 = to_next[0] = from[0];
@@ -357,30 +358,24 @@ lcp_xc_inline (vlib_main_t *vm, vlib_node_runtime_t *node, vlib_frame_t *frame,
vlib_buffer_advance (b0, -lip->lip_rewrite_len);
eth = vlib_buffer_get_current (b0);
- if (ethernet_address_cast (eth->dst_address))
- ai = lip->lip_phy_adjs.adj_index[af];
- else
+ ai = ADJ_INDEX_INVALID;
+ if (!ethernet_address_cast (eth->dst_address))
ai = lcp_adj_lkup ((u8 *) eth, lip->lip_rewrite_len,
vnet_buffer (b0)->sw_if_index[VLIB_TX]);
+ if (ai == ADJ_INDEX_INVALID)
+ ai = lip->lip_phy_adjs.adj_index[af];
- if (ADJ_INDEX_INVALID != ai)
- {
- const ip_adjacency_t *adj;
-
- adj = adj_get (ai);
- vnet_buffer (b0)->ip.adj_index[VLIB_TX] = ai;
- next0 = adj->rewrite_header.next_index;
- vnet_buffer (b0)->ip.save_rewrite_length = lip->lip_rewrite_len;
-
- if (PREDICT_FALSE (adj->rewrite_header.flags &
- VNET_REWRITE_HAS_FEATURES))
- vnet_feature_arc_start_w_cfg_index (
- lm->output_feature_arc_index,
- vnet_buffer (b0)->sw_if_index[VLIB_TX], &next0, b0,
- adj->ia_cfg_index);
- }
- else
- next0 = LCP_XC_NEXT_DROP;
+ adj = adj_get (ai);
+ vnet_buffer (b0)->ip.adj_index[VLIB_TX] = ai;
+ next0 = adj->rewrite_header.next_index;
+ vnet_buffer (b0)->ip.save_rewrite_length = lip->lip_rewrite_len;
+
+ if (PREDICT_FALSE (adj->rewrite_header.flags &
+ VNET_REWRITE_HAS_FEATURES))
+ vnet_feature_arc_start_w_cfg_index (
+ lm->output_feature_arc_index,
+ vnet_buffer (b0)->sw_if_index[VLIB_TX], &next0, b0,
+ adj->ia_cfg_index);
if (PREDICT_FALSE ((b0->flags & VLIB_BUFFER_IS_TRACED)))
{