From 172e698bb8239d0060d9d494adfba928507f95b2 Mon Sep 17 00:00:00 2001 From: Ivan Nardi <12729895+IvanNardi@users.noreply.github.com> Date: Thu, 28 Jul 2022 12:39:18 +0200 Subject: 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. --- tests/result/threema.pcap.out | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests/result/threema.pcap.out') diff --git a/tests/result/threema.pcap.out b/tests/result/threema.pcap.out index d6525d018..597a52b31 100644 --- a/tests/result/threema.pcap.out +++ b/tests/result/threema.pcap.out @@ -3,7 +3,7 @@ Guessed flow protos: 2 DPI Packets (TCP): 66 (11.00 pkts/flow) Confidence Match by IP : 2 (flows) Confidence DPI : 4 (flows) -Num dissector calls: 1262 (210.33 diss/flow) +Num dissector calls: 1244 (207.33 diss/flow) Threema 83 11578 6 -- cgit v1.2.3