From bd9578d2dc1bf4ed587e0158b8f3c0753fd18f29 Mon Sep 17 00:00:00 2001
From: "/C=EU/ST=EU/CN=Pablo Neira Ayuso/emailAddress=pablo@netfilter.org"
Date: Thu, 7 Jun 2007 15:37:44 +0000
Subject: fix nfct_snprintf behaviour if the buffer passed is too small
(similar to C99 convention)
---
src/conntrack/snprintf_xml.c | 142 +++++++++++--------------------------------
1 file changed, 34 insertions(+), 108 deletions(-)
(limited to 'src/conntrack/snprintf_xml.c')
diff --git a/src/conntrack/snprintf_xml.c b/src/conntrack/snprintf_xml.c
index 669d3e8..892c85b 100644
--- a/src/conntrack/snprintf_xml.c
+++ b/src/conntrack/snprintf_xml.c
@@ -67,12 +67,6 @@ enum {
__ADDR_DST,
};
-static void buffer_size(int ret, unsigned int *size, unsigned int *len)
-{
- *size += ret;
- *len -= ret;
-}
-
static char *__proto2str(u_int8_t protonum)
{
return proto2str[protonum] ? proto2str[protonum] : "unknown";
@@ -123,45 +117,33 @@ static int __snprintf_addr_xml(char *buf,
switch(type) {
case __ADDR_SRC:
ret = snprintf(buf, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
break;
case __ADDR_DST:
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
break;
}
switch (tuple->l3protonum) {
case AF_INET:
ret = __snprintf_ipv4_xml(buf+size, len, tuple, type);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
break;
case AF_INET6:
ret = __snprintf_ipv6_xml(buf+size, len, tuple, type);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
break;
}
switch(type) {
case __ADDR_SRC:
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
break;
case __ADDR_DST:
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
break;
}
@@ -183,15 +165,11 @@ static int __snprintf_proto_xml(char *buf,
if (type == __ADDR_SRC) {
ret = snprintf(buf, len, "%u",
ntohs(tuple->l4src.tcp.port));
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
} else {
ret = snprintf(buf, len, "%u",
ntohs(tuple->l4dst.tcp.port));
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
break;
}
@@ -209,15 +187,11 @@ static int __snprintf_counters_xml(char *buf,
ret = snprintf(buf, len, "%llu",
ct->counters[type].packets);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = snprintf(buf+size, len, "%llu",
ct->counters[type].bytes);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
return size;
}
@@ -233,76 +207,50 @@ static int __snprintf_tuple_xml(char *buf,
ret = snprintf(buf, len, "",
dir == __DIR_ORIG ? "original" : "reply");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = snprintf(buf+size, len,
"",
tuple->l3protonum, __l3proto2str(tuple->l3protonum));
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_addr_xml(buf+size, len, tuple, __DIR_ORIG);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_addr_xml(buf+size, len, tuple, __DIR_REPL);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = snprintf(buf+size, len,
"",
tuple->protonum, __proto2str(tuple->protonum));
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_proto_xml(buf+size, len, tuple, __DIR_ORIG);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_proto_xml(buf+size, len, tuple, __DIR_REPL);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
if (test_bit(ATTR_ORIG_COUNTER_PACKETS, ct->set) &&
test_bit(ATTR_ORIG_COUNTER_BYTES, ct->set)) {
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_counters_xml(buf+size, len, ct, dir);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
return size;
}
@@ -331,19 +279,13 @@ int __snprintf_conntrack_xml(char *buf,
break;
}
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_tuple_xml(buf+size, len, ct, __DIR_ORIG);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
ret = __snprintf_tuple_xml(buf+size, len, ct, __DIR_REPL);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
if (test_bit(ATTR_TIMEOUT, ct->set) ||
test_bit(ATTR_MARK, ct->set) ||
@@ -351,47 +293,35 @@ int __snprintf_conntrack_xml(char *buf,
test_bit(ATTR_STATUS, ct->set)) {
ret = snprintf(buf+size, len,
"");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
if (test_bit(ATTR_TIMEOUT, ct->set)) {
ret = snprintf(buf+size, len,
"%u", ct->timeout);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
if (test_bit(ATTR_MARK, ct->set)) {
ret = snprintf(buf+size, len, "%u", ct->mark);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
if (test_bit(ATTR_USE, ct->set)) {
ret = snprintf(buf+size, len, "", ct->use);
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
if (test_bit(ATTR_STATUS, ct->set)
&& ct->status & IPS_ASSURED) {
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
if (test_bit(ATTR_STATUS, ct->set)
&& !(ct->status & IPS_SEEN_REPLY)) {
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
if (test_bit(ATTR_TIMEOUT, ct->set) ||
@@ -399,15 +329,11 @@ int __snprintf_conntrack_xml(char *buf,
test_bit(ATTR_USE, ct->set) ||
test_bit(ATTR_STATUS, ct->set)) {
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
}
ret = snprintf(buf+size, len, "");
- if (ret == -1)
- return -1;
- buffer_size(ret, &size, &len);
+ BUFFER_SIZE(ret, size, len);
return size;
}
--
cgit v1.2.3