diff options
| author | lns <matzeton@googlemail.com> | 2022-06-04 00:33:32 +0200 |
|---|---|---|
| committer | lns <matzeton@googlemail.com> | 2022-06-04 00:36:47 +0200 |
| commit | 51e8795bbbf94f411225e40479e3ad7e285b86e0 (patch) | |
| tree | 5b87d76bf6f34081776c233474ee94e0fd344459 /src/lib/protocols/syslog.c | |
| parent | 09fbe0a64a11b08a35435f516e9a19f7e0c20d7c (diff) | |
Fix syslog heap overflow introduced in 09fbe0a64a11b08a35435f516e9a19f7e0c20d7c.fix/syslog-heap-overflow
- fixes #1578
Signed-off-by: lns <matzeton@googlemail.com>
Diffstat (limited to 'src/lib/protocols/syslog.c')
| -rw-r--r-- | src/lib/protocols/syslog.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/lib/protocols/syslog.c b/src/lib/protocols/syslog.c index 406bf5e8f..866e0a0c0 100644 --- a/src/lib/protocols/syslog.c +++ b/src/lib/protocols/syslog.c @@ -68,7 +68,7 @@ void ndpi_search_syslog(struct ndpi_detection_module_struct NDPI_LOG_DBG2(ndpi_struct, "no blank following the >: do nothing\n"); } - while (i < packet->payload_packet_len) + while (i < packet->payload_packet_len - 1) { if (ndpi_isalnum(packet->payload[i]) == 0) { @@ -86,8 +86,7 @@ void ndpi_search_syslog(struct ndpi_detection_module_struct if (packet->payload[i] == ':') { - i++; - if (i >= packet->payload_packet_len || + if (++i >= packet->payload_packet_len || packet->payload[i] != ' ') { NDPI_EXCLUDE_PROTO(ndpi_struct, flow); |