diff options
| author | Ivan Nardi <12729895+IvanNardi@users.noreply.github.com> | 2022-07-28 12:39:18 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-07-28 12:39:18 +0200 |
| commit | 172e698bb8239d0060d9d494adfba928507f95b2 (patch) | |
| tree | f5637b5f324a99d7186421daa57cd99a337db798 /src/include | |
| parent | d8d525fff2dc34db62de2598767046de813e4f0d (diff) | |
TINC: avoid processing SYN packets (#1676)
Since e6b332aa, we have proper support for detecting client/server
direction. So Tinc dissector is now able to properly initialize the
cache entry only when needed and not anymore at the SYN time; initializing
that entry for **every** SYN packets was a complete waste of resources.
Since 4896dabb, the various `struct ndpi_call_function_struct`
structures are not more separate objects and therefore comparing them
using only their pointers is bogus: this bug was triggered by this
change because `ndpi_str->callback_buffer_size_tcp_no_payload` is now 0.
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/ndpi_typedefs.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/include/ndpi_typedefs.h b/src/include/ndpi_typedefs.h index 9928ce321..dde19c488 100644 --- a/src/include/ndpi_typedefs.h +++ b/src/include/ndpi_typedefs.h @@ -556,11 +556,6 @@ struct ndpi_flow_input_info { /* ******************* ********************* ****************** */ /* ************************************************************ */ -PACK_ON struct tinc_cache_entry { - u_int32_t src_address; - u_int32_t dst_address; - u_int16_t dst_port; -} PACK_OFF; //CFFI.NDPI_PACKED_STRUCTURES #endif // NDPI_CFFI_PREPROCESSING_EXCLUDE_PACKED @@ -1458,7 +1453,6 @@ struct ndpi_flow_struct { /* NDPI_PROTOCOL_TINC */ u_int8_t tinc_state; - struct tinc_cache_entry tinc_cache_entry; /* Leave this field below at the end |