aboutsummaryrefslogtreecommitdiff
path: root/src/lib/protocols/stun.c
diff options
context:
space:
mode:
authorLuca Deri <deri@ntop.org>2021-01-13 16:46:51 +0100
committerLuca Deri <deri@ntop.org>2021-01-13 16:46:51 +0100
commite9f43516ec06638783fd03dea3aba0112584f566 (patch)
treef166782e3b2911ed1f92c5cb85a6a21b7183343a /src/lib/protocols/stun.c
parent68b6ac7da8b2f3c5f0ffa3f29015eefaa37e1ff5 (diff)
Improves STUN dissection removing an invalid termination condition that prevented Skype calls to be properly identified
Diffstat (limited to 'src/lib/protocols/stun.c')
-rw-r--r--src/lib/protocols/stun.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/src/lib/protocols/stun.c b/src/lib/protocols/stun.c
index fe2030ffb..cc1836a9b 100644
--- a/src/lib/protocols/stun.c
+++ b/src/lib/protocols/stun.c
@@ -531,12 +531,13 @@ void ndpi_search_stun(struct ndpi_detection_module_struct *ndpi_struct, struct n
return;
}
+ if(flow->protos.stun_ssl.stun.num_udp_pkts >= MAX_NUM_STUN_PKTS)
+ NDPI_EXCLUDE_PROTO(ndpi_struct, flow);
+
if(flow->packet_counter > 0) {
/* This might be a RTP stream: let's make sure we check it */
NDPI_CLR(&flow->excluded_protocol_bitmask, NDPI_PROTOCOL_RTP);
}
-
- NDPI_EXCLUDE_PROTO(ndpi_struct, flow);
}