diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2016-11-12 16:02:25 +0100 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2016-11-12 16:02:32 +0100 |
| commit | 800ad1737c7d284b579a369264fceca109b8a496 (patch) | |
| tree | 1f17fb1ed4d2bd9cf763de23de82efdb595fc5e9 | |
| parent | c7eb8d1082b5c7494a373b6b48403e3ae6c4cce3 (diff) | |
COAP: fix typo about checking twice source port
Found via Clang Analyzer (Used d_port variable)
| -rw-r--r-- | src/lib/protocols/coap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/lib/protocols/coap.c b/src/lib/protocols/coap.c index 5f8e97863..252ffc9c4 100644 --- a/src/lib/protocols/coap.c +++ b/src/lib/protocols/coap.c @@ -115,7 +115,7 @@ void ndpi_search_coap (struct ndpi_detection_module_struct *ndpi_struct, u_int16_t s_port = ntohs(flow->packet.udp->source); u_int16_t d_port = ntohs(flow->packet.udp->dest); - if((!isCoAPport(s_port) && !isCoAPport(s_port)) + if((!isCoAPport(s_port) && !isCoAPport(d_port)) || (packet->payload_packet_len < 4) // header too short ) { NDPI_LOG(NDPI_PROTOCOL_COAP, ndpi_struct, NDPI_LOG_DEBUG, "excluding Coap\n"); |