diff options
| author | Toni Uhlig <matzeton@googlemail.com> | 2020-09-02 00:12:15 +0200 |
|---|---|---|
| committer | Toni Uhlig <matzeton@googlemail.com> | 2020-09-02 07:29:33 +0200 |
| commit | 580859a47d164b64c95de2da6df2d49332af4875 (patch) | |
| tree | d62cfc51567e5aaa7633b8843a564d782fa2e691 /src/lib/protocols | |
| parent | 5d2551ff3e8544cc76e9b6a2747b95db8c7f97d5 (diff) | |
Fixed false positive detection for Skype.SkypeCall (affects at least Cisco HSRP and RADIUS).
Signed-off-by: Toni Uhlig <matzeton@googlemail.com>
Diffstat (limited to 'src/lib/protocols')
| -rw-r--r-- | src/lib/protocols/skype.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/lib/protocols/skype.c b/src/lib/protocols/skype.c index dfc3f274f..01d93bda7 100644 --- a/src/lib/protocols/skype.c +++ b/src/lib/protocols/skype.c @@ -64,13 +64,14 @@ static void ndpi_check_skype(struct ndpi_detection_module_struct *ndpi_struct, s || (((packet->payload[0] & 0xF0) >> 4) == 0x07 /* Skype */) ) && (packet->payload[0] != 0x30) /* Avoid invalid SNMP detection */ - && (packet->payload[0] != 0x0) /* Avoid invalid CAPWAP detection */ + && (packet->payload[0] != 0x00) /* Avoid invalid CAPWAP detection */ && (packet->payload[2] == 0x02))) { - if(is_port(sport, dport, 8801)) + if(is_port(sport, dport, 8801)) { ndpi_set_detected_protocol(ndpi_struct, flow, NDPI_PROTOCOL_ZOOM, NDPI_PROTOCOL_UNKNOWN); - else + } else if (payload_len >= 16 && packet->payload[0] != 0x01) /* Avoid invalid Cisco HSRP detection / RADIUS */ { ndpi_set_detected_protocol(ndpi_struct, flow, NDPI_PROTOCOL_SKYPE_CALL, NDPI_PROTOCOL_SKYPE); + } } } |