Skip to contentSkip to conversation

libnl: nfqueue: add nfqueue specific socket allocation function

PATCH Applied Patrick McHardy 2 participants 2 messages Oct 23, 2008

Subsystems (3)
Networking Network drivers netfilter
lore
Share & tools
lore

Patrick McHardy

author
#0 lore raw

commit e92539843a0c7e5116254382626cce226bf2135e

Author: Patrick McHardy <[email protected]>
Date:   Thu Oct 23 13:46:16 2008 +0200

libnl: nfqueue: add nfqueue specific socket allocation function

nfqueue users usually send verdict messages from the receive callback. When waiting for ACKs, the receive callback might be called again recursively until the stack blows up.

Add a nfqueue specific socket allocation function that automatically disables ACKing for the socket.

Signed-off-by Patrick McHardy <[email protected]>
include/netlink/netfilter/queue.h +2 -0
index 491b361..664610d 100644
@@ -31,6 +31,8 @@ enum nfnl_queue_copy_mode {
 };
 
 /* General */
+extern struct nl_sock *		nfnl_queue_socket_alloc(void);
+
 extern struct nfnl_queue *	nfnl_queue_alloc(void);
 
 extern void			nfnl_queue_get(struct nfnl_queue *);
lib/netfilter/queue.c +10 -0
index 8d3c7ce..ff1de0e 100644
@@ -24,6 +24,16 @@
 #include <netlink/netfilter/nfnl.h>
 #include <netlink/netfilter/queue.h>
 
+struct nl_sock *nfnl_queue_socket_alloc(void)
+{
+	struct nl_sock *nlsk;
+
+	nlsk = nl_socket_alloc();
+	if (nlsk)
+		nl_socket_disable_auto_ack(nlsk);
+	return nlsk;
+}
+
 static int send_queue_request(struct nl_sock *sk, struct nl_msg *msg)
 {
 	int err;
src/nf-queue.c +4 -1
index 4e17f70..0ed9e69 100644
@@ -47,7 +47,10 @@ int main(int argc, char *argv[])
 	int err = 1;
 	int family;
 
-	nf_sock = nlt_alloc_socket();
+	nf_sock = nfnl_queue_socket_alloc();
+	if (nf_sock == NULL)
+		fatal(ENOBUFS, "Unable to allocate netlink socket");
+
 	nl_disable_sequence_check(nf_sock);
 	nl_socket_modify_cb(nf_sock, NL_CB_VALID, NL_CB_CUSTOM, event_input, NULL);
Applied
#1 lore raw

* Patrick McHardy <[email protected]> 2008-10-23 13:47

Quoted 6 lines

commit e92539843a0c7e5116254382626cce226bf2135e

Author: Patrick McHardy <[email protected]>
Date:   Thu Oct 23 13:46:16 2008 +0200

libnl: nfqueue: add nfqueue specific socket allocation function

Quoted 8 lines

nfqueue users usually send verdict messages from the receive callback. When waiting for ACKs, the receive callback might be called again recursively until the stack blows up.

Add a nfqueue specific socket allocation function that automatically disables ACKing for the socket.

Signed-off-by Patrick McHardy <[email protected]>

Applied, thanks Patrick.