It seems that there is nothing i can really add to there except maybe: diff --git a/src/nlmsg.c b/src/nlmsg.c index aebdd5e..cabd8be 100644 --- a/src/nlmsg.c +++ b/src/nlmsg.c @@ -137,6 +137,7 @@ static int nfq_pkt_parse_attr_cb(const struct nlattr *attr, void *data) case NFQA_IFINDEX_PHYSOUTDEV: case NFQA_CAP_LEN: case NFQA_SKB_INFO: + case NFQA_SECCTX: case NFQA_UID: case NFQA_GID: if (mnl_attr_validate(attr, MNL_TYPE_U32) < 0) but that's a one line, will this be sufficient ? if so i'll merge this with the previous libnetfilter_queue patch and send it as one. On 06/16/2015 02:37 PM, Pablo Neira Ayuso wrote: > On Tue, Jun 16, 2015 at 02:25:13PM +0200, Roman Kubiak wrote: >> This patch is an addition to "[PATCH v3] nfnetlink_queue: add security context information" >> It adds and example to libmnl that illustrates how to fetch security context. >> A corresponding patch was sent for libnetfilter_queue already. >> >> -- cut here >> >> This patch modifies the example program for nf-queue >> to demonstrate how to retriece security context information >> for queued packages. This can also be easily extended to >> retrieve other information supported by this subsystem. > > This extension for the libmnl example is fine. > > However, when we asked for changes, we actually mean that you enhance: > > libnetfilter_queue/src/nlmsg.c > > to support this. Thanks. > > -- -------------- Roman Kubiak -------------- -- 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