[PATCH 4/4] utils: resolve compiler warnings

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
---
 utils/nfulnl_test.c |    4 ++--
 utils/ulog_test.c   |    4 ++--
 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);
 		}
 	}
-- 
1.7.1

--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux