wext: Pull top-level ioctl dispatch logic into helper function.
authorDavid S. Miller <davem@davemloft.net>
Tue, 3 Jun 2008 14:36:30 +0000 (07:36 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 17 Jun 2008 01:32:09 +0000 (18:32 -0700)
Signed-off-by: David S. Miller <davem@davemloft.net>
net/wireless/wext.c

index e9c88172ec55f951a39c1ec103b4a8a9b470750e..09022cbb58ba5d6789351d3905e08818f43b6c90 100644 (file)
@@ -1079,8 +1079,10 @@ static int wext_permission_check(unsigned int cmd)
 }
 
 /* entry point from dev ioctl */
-int wext_handle_ioctl(struct net *net, struct ifreq *ifr, unsigned int cmd,
-                     void __user *arg)
+static int wext_ioctl_dispatch(struct net *net, struct ifreq *ifr,
+                              unsigned int cmd,
+                              wext_ioctl_func standard,
+                              wext_ioctl_func private)
 {
        int ret = wext_permission_check(cmd);
 
@@ -1089,12 +1091,24 @@ int wext_handle_ioctl(struct net *net, struct ifreq *ifr, unsigned int cmd,
 
        dev_load(net, ifr->ifr_name);
        rtnl_lock();
-       ret = wireless_process_ioctl(net, ifr, cmd,
-                                    ioctl_standard_call,
-                                    ioctl_private_call);
+       ret = wireless_process_ioctl(net, ifr, cmd, standard, private);
        rtnl_unlock();
-       if (IW_IS_GET(cmd) && copy_to_user(arg, ifr, sizeof(struct iwreq)))
+
+       return ret;
+}
+
+int wext_handle_ioctl(struct net *net, struct ifreq *ifr, unsigned int cmd,
+                     void __user *arg)
+{
+       int ret = wext_ioctl_dispatch(net, ifr, cmd,
+                                     ioctl_standard_call,
+                                     ioctl_private_call);
+
+       if (ret >= 0 &&
+           IW_IS_GET(cmd) &&
+           copy_to_user(arg, ifr, sizeof(struct iwreq)))
                return -EFAULT;
+
        return ret;
 }