diff options
| author | Thomas Haller <thaller@redhat.com> | 2023-08-29 14:53:35 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2023-08-29 19:56:03 +0200 |
| commit | 05acd523d1633223648d3bec35b002d5eed6fbb2 (patch) | |
| tree | ab85b53f8686208be710ae13d531dc44ef1173cc /src | |
| parent | e76912b0ec900107b99dffe8fd1c37a845f708b3 (diff) | |
evaluate: don't needlessly clear full string buffer in stmt_evaluate_log_prefix()
Signed-off-by: Thomas Haller <thaller@redhat.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/evaluate.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/evaluate.c b/src/evaluate.c index 83c4b045..c13be824 100644 --- a/src/evaluate.c +++ b/src/evaluate.c @@ -4125,7 +4125,8 @@ static int stmt_evaluate_queue(struct eval_ctx *ctx, struct stmt *stmt) static int stmt_evaluate_log_prefix(struct eval_ctx *ctx, struct stmt *stmt) { - char prefix[NF_LOG_PREFIXLEN] = {}, tmp[NF_LOG_PREFIXLEN] = {}; + char tmp[NF_LOG_PREFIXLEN] = {}; + char prefix[NF_LOG_PREFIXLEN]; size_t len = sizeof(prefix); size_t offset = 0; struct expr *expr; @@ -4133,6 +4134,8 @@ static int stmt_evaluate_log_prefix(struct eval_ctx *ctx, struct stmt *stmt) if (stmt->log.prefix->etype != EXPR_LIST) return 0; + prefix[0] = '\0'; + list_for_each_entry(expr, &stmt->log.prefix->expressions, list) { int ret; |