summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJan Engelhardt <jengelh@medozas.de>2010-12-30 23:03:28 +0100
committerJan Engelhardt <jengelh@medozas.de>2010-12-30 23:07:25 +0100
commit3e456f175a6e25e30e60934ff08c7108b1f86654 (patch)
tree0b704f6e89ad13e54ff0bf4d9968b017139c20fe
parent6f236834592023b7c40e5cca51eb07d7c8202f3c (diff)
utils: resolve compiler warnings
make nfulnl_test ulog_test make[1]: Entering directory `/home/jengelh/code/libnetfilter_log/utils' CC nfulnl_test.o nfulnl_test.c: In function "cb": nfulnl_test.c:47:1: warning: no return statement in function returning non-void nfulnl_test.c: In function "main": nfulnl_test.c:102:20: warning: unused variable "nlh" CCLD nfulnl_test CC ulog_test.o ulog_test.c: In function "handle_packet": ulog_test.c:27:9: warning: format "%d" expects type ‘int’, but argument 4 has type ‘size_t’ ulog_test.c: In function "main": ulog_test.c:79:3: warning: suggest parentheses around assignment used as truth value CCLD ulog_test make[1]: Leaving directory `/home/jengelh/code/libnetfilter_log/utils' Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
-rw-r--r--utils/nfulnl_test.c4
-rw-r--r--utils/ulog_test.c4
2 files changed, 4 insertions, 4 deletions
diff --git a/utils/nfulnl_test.c b/utils/nfulnl_test.c
index 139013d..ae5da79 100644
--- a/utils/nfulnl_test.c
+++ b/utils/nfulnl_test.c
@@ -44,6 +44,7 @@ static int cb(struct nflog_g_handle *gh, struct nfgenmsg *nfmsg,
struct nflog_data *nfa, void *data)
{
print_pkt(nfa);
+ return 0;
}
@@ -99,7 +100,6 @@ int main(int argc, char **argv)
printf("going into main loop\n");
while ((rv = recv(fd, buf, sizeof(buf), 0)) && rv >= 0) {
- struct nlmsghdr *nlh;
printf("pkt received (len=%u)\n", rv);
/* handle messages in just-received packet */
@@ -121,5 +121,5 @@ int main(int argc, char **argv)
printf("closing handle\n");
nflog_close(h);
- exit(0);
+ return EXIT_SUCCESS;
}
diff --git a/utils/ulog_test.c b/utils/ulog_test.c
index d568aa4..f3adec2 100644
--- a/utils/ulog_test.c
+++ b/utils/ulog_test.c
@@ -23,7 +23,7 @@ void handle_packet(ulog_packet_msg_t *pkt)
unsigned char *p;
int i;
- printf("Hook=%u Mark=%lu len=%d ",
+ printf("Hook=%u Mark=%lu len=%zu ",
pkt->hook, pkt->mark, pkt->data_len);
if (strlen(pkt->prefix))
printf("Prefix=%s ", pkt->prefix);
@@ -76,7 +76,7 @@ int main(int argc, char *argv[])
exit(1);
}
printf("%d bytes received\n", len);
- while (upkt = ipulog_get_packet(h, buf, len)) {
+ while ((upkt = ipulog_get_packet(h, buf, len)) != NULL) {
handle_packet(upkt);
}
}