diff options
| author |   Neale Ranns <nranns@cisco.com> | 2019-06-28 08:33:24 +0000 |
|---|---|---|
| committer | Neale Ranns <nranns@cisco.com> | 2019-06-28 08:33:24 +0000 |
| commit | 4752b29cfe229806c1cb2ae852c188d36be4c7cf (patch) | |
| tree | 7665d6f8d0b6ac83268fb377792e87be01e9dfb6 /src | |
| parent | 4f731f829f29c5579257ecf1ef4b129a9d79fac8 (diff) | |
igmp: Trace more data form input packets
Type: feature
Change-Id: I40ecc0da1281115f6e540064224538fe9472887e
Signed-off-by: Neale Ranns <nranns@cisco.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/plugins/igmp/igmp_error.h | 1 | ||||
| -rw-r--r-- | src/plugins/igmp/igmp_input.c | 22 |
2 files changed, 16 insertions, 7 deletions
diff --git a/src/plugins/igmp/igmp_error.h b/src/plugins/igmp/igmp_error.h index fbd0dc434f0..d3b80f73169 100644 --- a/src/plugins/igmp/igmp_error.h +++ b/src/plugins/igmp/igmp_error.h @@ -23,6 +23,7 @@ _ (UNSPECIFIED, "unspecified error") \ _ (INVALID_PROTOCOL, "invalid ip4 protocol") \ _ (BAD_CHECKSUM, "bad checksum") \ + _ (BAD_LENGTH, "bad length") \ _ (UNKNOWN_TYPE, "unknown igmp message type") \ _ (NOT_ENABLED, "IGMP not enabled on this interface") \ diff --git a/src/plugins/igmp/igmp_input.c b/src/plugins/igmp/igmp_input.c index d5f714bc033..a49790fe119 100644 --- a/src/plugins/igmp/igmp_input.c +++ b/src/plugins/igmp/igmp_input.c @@ -61,7 +61,7 @@ typedef struct { u32 next_index; u32 sw_if_index; - + u32 len; u8 packet_data[64]; } igmp_input_trace_t; @@ -100,10 +100,12 @@ format_igmp_parse_query_trace (u8 * s, va_list * va) CLIB_UNUSED (vlib_node_t * node) = va_arg (*va, vlib_node_t *); igmp_input_trace_t *t = va_arg (*va, igmp_input_trace_t *); - s = format (s, "sw_if_index %u next-input %u", - t->sw_if_index, t->next_index); + s = format (s, "sw_if_index %u next-input %u len %u", + t->sw_if_index, t->next_index, t->len); s = format (s, "\n%U", format_igmp_query_v3, t->packet_data, sizeof (t->packet_data)); + s = format (s, "\n%U", format_hex_bytes, + t->packet_data, sizeof (t->packet_data)); return s; } @@ -203,6 +205,7 @@ igmp_input (vlib_main_t * vm, vlib_node_runtime_t * node, tr = vlib_add_trace (vm, node, b, sizeof (*tr)); tr->next_index = next; tr->sw_if_index = vnet_buffer (b)->sw_if_index[VLIB_RX]; + tr->len = vlib_buffer_length_in_chain (vm, b); clib_memcpy_fast (tr->packet_data, vlib_buffer_get_current (b), sizeof (tr->packet_data)); } @@ -280,6 +283,7 @@ igmp_parse_query (vlib_main_t * vm, vlib_node_runtime_t * node, tr = vlib_add_trace (vm, node, b, sizeof (*tr)); tr->next_index = next; tr->sw_if_index = vnet_buffer (b)->sw_if_index[VLIB_RX]; + tr->len = vlib_buffer_length_in_chain (vm, b); clib_memcpy_fast (tr->packet_data, vlib_buffer_get_current (b), sizeof (tr->packet_data)); } @@ -302,10 +306,14 @@ igmp_parse_query (vlib_main_t * vm, vlib_node_runtime_t * node, vl_api_rpc_call_main_thread (igmp_handle_query, (u8 *) args, sizeof (*args) + len); } - /* - * else a packet that is reporting more or less sources - * than it really has, bin it - */ + else + { + /* + * else a packet that is reporting more or less sources + * than it really has, bin it + */ + b->error = node->errors[IGMP_ERROR_BAD_LENGTH]; + } vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next, n_left_to_next, bi, next); |