diff options
author | tianzhishuizhu <tianzhishuizhu5@gamil.com> | 2016-09-13 15:51:17 +0800 |
---|---|---|
committer | tianzhishuizhu <tianzhishuizhu5@gamil.com> | 2016-09-13 15:51:17 +0800 |
commit | 2b67b3bc810024ba2175d8045e67a64fdbb9de59 (patch) | |
tree | ce8ddf6f4328d2a2cba490bd483c659f3f0e5e93 /src/lib/protocols/syslog.c | |
parent | c10052764b658826ba4da73e11e6ff153d4bc916 (diff) |
improve readability
Diffstat (limited to 'src/lib/protocols/syslog.c')
-rw-r--r-- | src/lib/protocols/syslog.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/lib/protocols/syslog.c b/src/lib/protocols/syslog.c index ffd897221..3f8a6c0b8 100644 --- a/src/lib/protocols/syslog.c +++ b/src/lib/protocols/syslog.c @@ -46,15 +46,14 @@ void ndpi_search_syslog(struct ndpi_detection_module_struct if (packet->payload_packet_len > 20 && packet->payload_packet_len <= 1024 && packet->payload[0] == '<') { NDPI_LOG(NDPI_PROTOCOL_SYSLOG, ndpi_struct, NDPI_LOG_DEBUG, "checked len>20 and <1024 and first symbol=<.\n"); - i = 1; - for (;;) { - if (packet->payload[i] < '0' || packet->payload[i] > '9' || i++ > 3) { - NDPI_LOG(NDPI_PROTOCOL_SYSLOG, ndpi_struct, NDPI_LOG_DEBUG, - "read symbols while the symbol is a number.\n"); - break; + for (i = 1; i <= 3; i++) { + if (packet->payload[i] < '0' || packet->payload[i] > '9') { + break; } } + NDPI_LOG(NDPI_PROTOCOL_SYSLOG, ndpi_struct, NDPI_LOG_DEBUG, + "read symbols while the symbol is a number.\n"); if (packet->payload[i++] != '>') { NDPI_LOG(NDPI_PROTOCOL_SYSLOG, ndpi_struct, NDPI_LOG_DEBUG, "there is no > following the number.\n"); |