path: root/utils
diff options
authorDavid Fabro <>2010-06-10 15:03:44 +0200
committerPablo Neira Ayuso <>2010-06-10 15:03:44 +0200
commit21fd1834b5ce0a1f5b590f7e1ad23bba64fbafdf (patch)
tree638d843052af388d42bdbd9419ed36c10f4c5b66 /utils
parent6e204c91b6c7ece751331de2fa3fb460bc5edd59 (diff)
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 <>
Diffstat (limited to 'utils')
1 files changed, 1 insertions, 1 deletions
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) {