From: Victor Julien Date: Sat, 26 Mar 2016 10:28:30 +0000 (+0100) Subject: stats-log: fix layout issue due to decoder stats X-Git-Tag: suricata-3.0.1~25 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=400fa5b9e5fab1bad4e78ab72e9ba8cda6ccd7e8;p=thirdparty%2Fsuricata.git stats-log: fix layout issue due to decoder stats --- diff --git a/src/log-stats.c b/src/log-stats.c index ad6ee54720..ae7ed50605 100644 --- a/src/log-stats.c +++ b/src/log-stats.c @@ -90,17 +90,17 @@ int LogStatsLogger(ThreadVars *tv, void *thread_data, const StatsTable *st) int days = in_hours / 24; MemBufferWriteString(aft->buffer, "----------------------------------------------" - "---------------------\n"); + "--------------------------------------\n"); MemBufferWriteString(aft->buffer, "Date: %" PRId32 "/%" PRId32 "/%04d -- " "%02d:%02d:%02d (uptime: %"PRId32"d, %02dh %02dm %02ds)\n", tms->tm_mon + 1, tms->tm_mday, tms->tm_year + 1900, tms->tm_hour, tms->tm_min, tms->tm_sec, days, hours, min, sec); MemBufferWriteString(aft->buffer, "----------------------------------------------" - "---------------------\n"); - MemBufferWriteString(aft->buffer, "%-25s | %-25s | %-s\n", "Counter", "TM Name", + "--------------------------------------\n"); + MemBufferWriteString(aft->buffer, "%-42s | %-25s | %-s\n", "Counter", "TM Name", "Value"); MemBufferWriteString(aft->buffer, "----------------------------------------------" - "---------------------\n"); + "--------------------------------------\n"); /* global stats */ uint32_t u = 0; @@ -112,8 +112,8 @@ int LogStatsLogger(ThreadVars *tv, void *thread_data, const StatsTable *st) if (!(aft->statslog_ctx->flags & LOG_STATS_NULLS) && st->stats[u].value == 0) continue; - char line[1024]; - size_t len = snprintf(line, sizeof(line), "%-25s | %-25s | %-" PRIu64 "\n", + char line[256]; + size_t len = snprintf(line, sizeof(line), "%-42s | %-25s | %-" PRIu64 "\n", st->stats[u].name, st->stats[u].tm_name, st->stats[u].value); /* since we can have many threads, the buffer might not be big enough. @@ -138,8 +138,8 @@ int LogStatsLogger(ThreadVars *tv, void *thread_data, const StatsTable *st) if (st->tstats[u].name == NULL) continue; - char line[1024]; - size_t len = snprintf(line, sizeof(line), "%-25s | %-25s | %-" PRIu64 "\n", + char line[256]; + size_t len = snprintf(line, sizeof(line), "%-42s | %-25s | %-" PRIu64 "\n", st->tstats[u].name, st->tstats[u].tm_name, st->tstats[u].value); /* since we can have many threads, the buffer might not be big enough.