summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorPablo Neira Ayuso <pablo@netfilter.org>2015-01-04 15:20:41 +0100
committerPablo Neira Ayuso <pablo@netfilter.org>2015-01-06 22:27:47 +0100
commit62924af247e95de7041a6d6f2d06cdd05152e2dc (patch)
tree6397178a16dde74968842feed089a895c8cba52e /net
parent9ea2aa8b7dba9e99544c4187cc298face254569f (diff)
downloadop-kernel-dev-62924af247e95de7041a6d6f2d06cdd05152e2dc.zip
op-kernel-dev-62924af247e95de7041a6d6f2d06cdd05152e2dc.tar.gz
netfilter: nfnetlink: relax strict multicast group check from netlink_bind
Relax the checking that was introduced in 97840cb ("netfilter: nfnetlink: fix insufficient validation in nfnetlink_bind") when the subscription bitmask is used. Existing userspace code code may request to listen to all of the existing netlink groups by setting an all to one subscription group bitmask. Netlink already validates subscription via setsockopt() for us. Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'net')
-rw-r--r--net/netfilter/nfnetlink.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/netfilter/nfnetlink.c b/net/netfilter/nfnetlink.c
index c6619d4..1aa7049 100644
--- a/net/netfilter/nfnetlink.c
+++ b/net/netfilter/nfnetlink.c
@@ -470,7 +470,7 @@ static int nfnetlink_bind(int group)
int type;
if (group <= NFNLGRP_NONE || group > NFNLGRP_MAX)
- return -EINVAL;
+ return 0;
type = nfnl_group2type[group];
OpenPOWER on IntegriCloud