mac802154: Avoid rtnl deadlock in mac802154_wpan_ioctl().
authorLennert Buytenhek <buytenh@wantstofly.org>
Tue, 26 May 2015 12:06:10 +0000 (15:06 +0300)
committerMarcel Holtmann <marcel@holtmann.org>
Tue, 26 May 2015 16:03:01 +0000 (18:03 +0200)
->ndo_do_ioctl() can be entered with the rtnl lock already held,
for example when sending a wext ioctl to a device (in which case
the rtnl lock is taken by wext_ioctl_dispatch()), but
mac802154_wpan_ioctl() currently unconditionally takes the rtnl
lock on entry, which can cause deadlocks.

To fix this, bail out of mac802154_wpan_ioctl() before taking the
rtnl lock if the ioctl cmd is not one of the cmds we implement.

Signed-off-by: Lennert Buytenhek <buytenh@wantstofly.org>
Acked-by: Alexander Aring <alex.aring@gmail.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/mac802154/iface.c

index f30788d2702fa2e77d5f08111af7961dcc155a6e..b544b5dc4bfbd1968372c65dee771d0b59cb9671 100644 (file)
@@ -62,6 +62,9 @@ mac802154_wpan_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
                (struct sockaddr_ieee802154 *)&ifr->ifr_addr;
        int err = -ENOIOCTLCMD;
 
+       if (cmd != SIOCGIFADDR && cmd != SIOCSIFADDR)
+               return err;
+
        rtnl_lock();
 
        switch (cmd) {