diff options
author | theirix <theirix@gmail.com> | 2016-04-12 22:14:13 +0300 |
---|---|---|
committer | theirix <theirix@gmail.com> | 2016-04-12 22:14:13 +0300 |
commit | c22d3d3cae8fc6d5fcf1b7320a7602426a9b9ca2 (patch) | |
tree | adebbf469efc03bba8a8d955b5116b35ae50ef3e /src/lib/protocols/radius.c | |
parent | fb3fc0c6de201a2ab34b6f7ce4d5dfc2c54c3b5e (diff) |
Fixed more buffer overflows with small packets
Diffstat (limited to 'src/lib/protocols/radius.c')
-rw-r--r-- | src/lib/protocols/radius.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/lib/protocols/radius.c b/src/lib/protocols/radius.c index 625dc4108..308049522 100644 --- a/src/lib/protocols/radius.c +++ b/src/lib/protocols/radius.c @@ -37,12 +37,11 @@ static void ndpi_check_radius(struct ndpi_detection_module_struct *ndpi_struct, if(packet->udp != NULL) { struct radius_header *h = (struct radius_header*)packet->payload; - u_int len = ntohs(h->len); if((payload_len > sizeof(struct radius_header)) && (h->code > 0) && (h->code <= 5) - && (len == payload_len)) { + && (ntohs(h->len) == payload_len)) { NDPI_LOG(NDPI_PROTOCOL_RADIUS, ndpi_struct, NDPI_LOG_DEBUG, "Found radius.\n"); ndpi_set_detected_protocol(ndpi_struct, flow, NDPI_PROTOCOL_RADIUS, NDPI_PROTOCOL_UNKNOWN); |