diff options
author | /C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org </C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org> | 2007-06-09 20:19:12 +0000 |
---|---|---|
committer | /C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org </C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org> | 2007-06-09 20:19:12 +0000 |
commit | 18a69966d7c2fb9e9ba28e9c0d540b06253379d1 (patch) | |
tree | 9ed51b411341f5e171335ab4170ace6748846aa0 | |
parent | ad922a36f5c5f92568055062b88d569928a27834 (diff) |
fix wrong snprintf return value checking: never returns -1
-rw-r--r-- | ulogd/extensions/ulogd_BASE.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ulogd/extensions/ulogd_BASE.c b/ulogd/extensions/ulogd_BASE.c index 0fd921c..412dc89 100644 --- a/ulogd/extensions/ulogd_BASE.c +++ b/ulogd/extensions/ulogd_BASE.c @@ -82,7 +82,7 @@ static ulog_iret_t *_interp_raw(ulog_interpreter_t *ip, for (i = 0; i < pkt->mac_len; i++, p++) { tmp = snprintf(ptr+len, siz-len, "%02x%s", *p, i==pkt->mac_len-1 ? "":":"); - if (tmp == -1) + if (tmp > siz-len) break; len += tmp; } |