From 070387d3f4b0b5a5742b95fca19404cb50fa9927 Mon Sep 17 00:00:00 2001 From: Klement Sekera Date: Tue, 2 Feb 2021 20:15:59 +0100 Subject: nat: improve type safety and remove unused param Type: improvement Change-Id: I456f9b14e6a4eb46c9c49f6e09acccae530e4ebc Signed-off-by: Klement Sekera --- src/plugins/nat/nat44-ei/nat44_ei_in2out.c | 47 +++++++++++++++--------------- 1 file changed, 24 insertions(+), 23 deletions(-) (limited to 'src/plugins/nat/nat44-ei/nat44_ei_in2out.c') diff --git a/src/plugins/nat/nat44-ei/nat44_ei_in2out.c b/src/plugins/nat/nat44-ei/nat44_ei_in2out.c index 6a4658513d7..ad0007bb8df 100644 --- a/src/plugins/nat/nat44-ei/nat44_ei_in2out.c +++ b/src/plugins/nat/nat44-ei/nat44_ei_in2out.c @@ -444,11 +444,11 @@ icmp_get_key (vlib_buffer_t * b, ip4_header_t * ip0, * @param e optional parameter */ u32 -icmp_match_in2out_slow (snat_main_t * sm, vlib_node_runtime_t * node, - u32 thread_index, vlib_buffer_t * b0, - ip4_header_t * ip0, ip4_address_t * addr, u16 * port, - u32 * fib_index, nat_protocol_t * proto, void *d, - void *e, u8 * dont_translate) +icmp_match_in2out_slow (snat_main_t *sm, vlib_node_runtime_t *node, + u32 thread_index, vlib_buffer_t *b0, ip4_header_t *ip0, + ip4_address_t *addr, u16 *port, u32 *fib_index, + nat_protocol_t *proto, snat_session_t **p_s0, + u8 *dont_translate) { snat_main_per_thread_data_t *tsm = &sm->per_thread_data[thread_index]; u32 sw_if_index0; @@ -542,8 +542,8 @@ out: *port = s0->out2in.port; *fib_index = s0->out2in.fib_index; } - if (d) - *(snat_session_t **) (d) = s0; + if (p_s0) + *p_s0 = s0; return next0; } #endif @@ -564,11 +564,11 @@ out: * @param e optional parameter */ u32 -icmp_match_in2out_fast (snat_main_t * sm, vlib_node_runtime_t * node, - u32 thread_index, vlib_buffer_t * b0, - ip4_header_t * ip0, ip4_address_t * addr, u16 * port, - u32 * fib_index, nat_protocol_t * proto, void *d, - void *e, u8 * dont_translate) +icmp_match_in2out_fast (snat_main_t *sm, vlib_node_runtime_t *node, + u32 thread_index, vlib_buffer_t *b0, ip4_header_t *ip0, + ip4_address_t *addr, u16 *port, u32 *fib_index, + nat_protocol_t *proto, snat_session_t **s0, + u8 *dont_translate) { u32 sw_if_index0; u8 is_addr_only; @@ -632,16 +632,17 @@ out: } #endif +u32 icmp_in2out (snat_main_t *sm, vlib_buffer_t *b0, ip4_header_t *ip0, + icmp46_header_t *icmp0, u32 sw_if_index0, u32 rx_fib_index0, + vlib_node_runtime_t *node, u32 next0, u32 thread_index, + snat_session_t **p_s0); + #ifndef CLIB_MARCH_VARIANT u32 -icmp_in2out (snat_main_t * sm, - vlib_buffer_t * b0, - ip4_header_t * ip0, - icmp46_header_t * icmp0, - u32 sw_if_index0, - u32 rx_fib_index0, - vlib_node_runtime_t * node, - u32 next0, u32 thread_index, void *d, void *e) +icmp_in2out (snat_main_t *sm, vlib_buffer_t *b0, ip4_header_t *ip0, + icmp46_header_t *icmp0, u32 sw_if_index0, u32 rx_fib_index0, + vlib_node_runtime_t *node, u32 next0, u32 thread_index, + snat_session_t **p_s0) { vlib_main_t *vm = vlib_get_main (); ip4_address_t addr; @@ -664,7 +665,7 @@ icmp_in2out (snat_main_t * sm, next0_tmp = sm->icmp_match_in2out_cb (sm, node, thread_index, b0, ip0, &addr, &port, - &fib_index, &protocol, d, e, &dont_translate); + &fib_index, &protocol, p_s0, &dont_translate); if (next0_tmp != ~0) next0 = next0_tmp; if (next0 == SNAT_IN2OUT_NEXT_DROP || dont_translate) @@ -806,7 +807,7 @@ icmp_in2out_slow_path (snat_main_t * sm, vlib_main_t *vm = vlib_get_main (); next0 = icmp_in2out (sm, b0, ip0, icmp0, sw_if_index0, rx_fib_index0, node, - next0, thread_index, p_s0, 0); + next0, thread_index, p_s0); snat_session_t *s0 = *p_s0; if (PREDICT_TRUE (next0 != SNAT_IN2OUT_NEXT_DROP && s0)) { @@ -1819,7 +1820,7 @@ VLIB_NODE_FN (snat_in2out_fast_node) (vlib_main_t * vm, if (PREDICT_FALSE (proto0 == NAT_PROTOCOL_ICMP)) { next0 = icmp_in2out (sm, b0, ip0, icmp0, sw_if_index0, - rx_fib_index0, node, next0, ~0, 0, 0); + rx_fib_index0, node, next0, ~0, 0); goto trace0; } -- cgit 1.2.3-korg