diff options
author | David Fabro <netfilter@meta-dynamic.com> | 2010-06-10 15:03:44 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2010-06-10 15:03:44 +0200 |
commit | 21fd1834b5ce0a1f5b590f7e1ad23bba64fbafdf (patch) | |
tree | 638d843052af388d42bdbd9419ed36c10f4c5b66 /include | |
parent | 6e204c91b6c7ece751331de2fa3fb460bc5edd59 (diff) | |
download | libnetfilter_queue-21fd1834b5ce0a1f5b590f7e1ad23bba64fbafdf.tar.gz |
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 <netfilter@meta-dynamic.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/libnetfilter_queue/libnetfilter_queue.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/libnetfilter_queue/libnetfilter_queue.h b/include/libnetfilter_queue/libnetfilter_queue.h index 2e2ca8b..53bda74 100644 --- a/include/libnetfilter_queue/libnetfilter_queue.h +++ b/include/libnetfilter_queue/libnetfilter_queue.h @@ -104,7 +104,7 @@ extern int nfq_get_physoutdev_name(struct nlif_handle *nlif_handle, extern struct nfqnl_msg_packet_hw *nfq_get_packet_hw(struct nfq_data *nfad); /* return -1 if problem, length otherwise */ -extern int nfq_get_payload(struct nfq_data *nfad, char **data); +extern int nfq_get_payload(struct nfq_data *nfad, unsigned char **data); #ifdef __cplusplus } /* extern "C" */ |