diff options
| author | Ivan Nardi <12729895+IvanNardi@users.noreply.github.com> | 2022-01-30 19:15:42 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-01-30 19:15:42 +0100 |
| commit | f4554218708e390299e0804c4d5d5fda70809b99 (patch) | |
| tree | 38260d4cb3673b70e5772eccf1421cd9a8069359 /fuzz/fuzz_process_packet.c | |
| parent | 513e386959e0d52c4b37cb704f6b0b7a128cc0e4 (diff) | |
Improve protocol stacks (#1425)
We should have two protocols in classification results only when the
"master" protocol allows some sub-protocols.
Classifications like `AmazonAWS`, `TLS/AmazonAWS`, `DNS/AmazonAWS` are
fine. However classifications like `NTP/Apple`, `BitTorrent/Azure`,
`DNScrypt.AmazonAWS` or `NestLogSink.Google` are misleading.
For example, `ndpiReader`shows `BitTorrent/Azure` flows under `Azure`
statistics; that seems to be wrong or, at least, very misleading.
This is quite important since we have lots of addresses from CDN
operators.
The only drawback of this solution is that right now ICMP traffic is
classified simply as `ICMP`; if we are really interested in ICMP stuff
we can restore the old behaviour later.
Diffstat (limited to 'fuzz/fuzz_process_packet.c')
0 files changed, 0 insertions, 0 deletions