diff options
| author | Ivan Nardi <12729895+IvanNardi@users.noreply.github.com> | 2023-01-17 08:26:42 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-01-17 08:26:42 +0100 |
| commit | ebb9ebd2a0a1536cb8f9d9dc510f52f33ed78eab (patch) | |
| tree | 6e3ee193e8e9e0e4ce1f2f9680ec252f1f46e8e2 /tests/result/iphone.pcap.out | |
| parent | 1f7c57deff9debbda3d26be906e067dcf73ce1f9 (diff) | |
Fix classification "by-port" (#1851)
Classification "by-port" should be the last possible effort, *after*
having test all the LRU caches.
Remove some dead code from `ndpi_detection_giveup()`:
`flow->guessed_protocol_id` is never set to any od those voip protocols
and at that point in this function we never have both a master *and* a
application protocols.
Coverage reports (both from unit tests and from fuzzing) confirms that
was dead code.
Diffstat (limited to 'tests/result/iphone.pcap.out')
| -rw-r--r-- | tests/result/iphone.pcap.out | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/result/iphone.pcap.out b/tests/result/iphone.pcap.out index 0354a9eea..1a549df85 100644 --- a/tests/result/iphone.pcap.out +++ b/tests/result/iphone.pcap.out @@ -7,13 +7,13 @@ Confidence Match by port : 1 (flows) Confidence DPI : 50 (flows) Num dissector calls: 344 (6.75 diss/flow) LRU cache ookla: 0/0/0 (insert/search/found) -LRU cache bittorrent: 0/0/0 (insert/search/found) +LRU cache bittorrent: 0/3/0 (insert/search/found) LRU cache zoom: 0/0/0 (insert/search/found) LRU cache stun: 0/0/0 (insert/search/found) LRU cache tls_cert: 0/0/0 (insert/search/found) LRU cache mining: 0/1/0 (insert/search/found) LRU cache msteams: 0/0/0 (insert/search/found) -LRU cache stun_zoom: 0/0/0 (insert/search/found) +LRU cache stun_zoom: 0/1/0 (insert/search/found) Automa host: 62/53 (search/found) Automa domain: 62/0 (search/found) Automa tls cert: 0/0 (search/found) |