From 21fd1834b5ce0a1f5b590f7e1ad23bba64fbafdf Mon Sep 17 00:00:00 2001 From: David Fabro Date: Thu, 10 Jun 2010 15:03:44 +0200 Subject: payload buffer datatypes are consistent The 'data' parameter to nfq_get_payload() returns pointer to unsigned char (rather than signed char) to make it consistent with the 'buf' parameter of nfq_set_verdict(), nfq_set_verdict2(), and nfq_set_verdict_mark(), all of which refer to the same data. Either signed or unsigned is fine, but they should be consistent as the output of nfq_get_payload() may be passed back into nfq_set_verdict*(); in that case, this change eliminates the need for typecasting in the calling code when using compilers that enforce strict typecasting. Signed-off-by: David Favro Signed-off-by: Pablo Neira Ayuso --- utils/nfqnl_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'utils/nfqnl_test.c') diff --git a/utils/nfqnl_test.c b/utils/nfqnl_test.c index 036bdab..9eebd9b 100644 --- a/utils/nfqnl_test.c +++ b/utils/nfqnl_test.c @@ -16,7 +16,7 @@ static u_int32_t print_pkt (struct nfq_data *tb) struct nfqnl_msg_packet_hw *hwph; u_int32_t mark,ifi; int ret; - char *data; + unsigned char *data; ph = nfq_get_msg_packet_hdr(tb); if (ph) { -- cgit v1.2.3