devinet_ioctl(): take copyin/copyout to caller
[muen/linux.git] / net / ipv4 / devinet.c
index 1771549d24385f3372852b6732a081c4328ceb0e..e056c0067f2cff22a89dd8f68cd4f0913529ad55 100644 (file)
@@ -946,11 +946,10 @@ static int inet_abc_len(__be32 addr)
 }
 
 
 }
 
 
-int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
+int devinet_ioctl(struct net *net, unsigned int cmd, struct ifreq *ifr)
 {
 {
-       struct ifreq ifr;
        struct sockaddr_in sin_orig;
        struct sockaddr_in sin_orig;
-       struct sockaddr_in *sin = (struct sockaddr_in *)&ifr.ifr_addr;
+       struct sockaddr_in *sin = (struct sockaddr_in *)&ifr->ifr_addr;
        struct in_device *in_dev;
        struct in_ifaddr **ifap = NULL;
        struct in_ifaddr *ifa = NULL;
        struct in_device *in_dev;
        struct in_ifaddr **ifap = NULL;
        struct in_ifaddr *ifa = NULL;
@@ -959,22 +958,16 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
        int ret = -EFAULT;
        int tryaddrmatch = 0;
 
        int ret = -EFAULT;
        int tryaddrmatch = 0;
 
-       /*
-        *      Fetch the caller's info block into kernel space
-        */
-
-       if (copy_from_user(&ifr, arg, sizeof(struct ifreq)))
-               goto out;
-       ifr.ifr_name[IFNAMSIZ - 1] = 0;
+       ifr->ifr_name[IFNAMSIZ - 1] = 0;
 
        /* save original address for comparison */
        memcpy(&sin_orig, sin, sizeof(*sin));
 
 
        /* save original address for comparison */
        memcpy(&sin_orig, sin, sizeof(*sin));
 
-       colon = strchr(ifr.ifr_name, ':');
+       colon = strchr(ifr->ifr_name, ':');
        if (colon)
                *colon = 0;
 
        if (colon)
                *colon = 0;
 
-       dev_load(net, ifr.ifr_name);
+       dev_load(net, ifr->ifr_name);
 
        switch (cmd) {
        case SIOCGIFADDR:       /* Get interface address */
 
        switch (cmd) {
        case SIOCGIFADDR:       /* Get interface address */
@@ -1014,7 +1007,7 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
        rtnl_lock();
 
        ret = -ENODEV;
        rtnl_lock();
 
        ret = -ENODEV;
-       dev = __dev_get_by_name(net, ifr.ifr_name);
+       dev = __dev_get_by_name(net, ifr->ifr_name);
        if (!dev)
                goto done;
 
        if (!dev)
                goto done;
 
@@ -1031,7 +1024,7 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
                           This is checked above. */
                        for (ifap = &in_dev->ifa_list; (ifa = *ifap) != NULL;
                             ifap = &ifa->ifa_next) {
                           This is checked above. */
                        for (ifap = &in_dev->ifa_list; (ifa = *ifap) != NULL;
                             ifap = &ifa->ifa_next) {
-                               if (!strcmp(ifr.ifr_name, ifa->ifa_label) &&
+                               if (!strcmp(ifr->ifr_name, ifa->ifa_label) &&
                                    sin_orig.sin_addr.s_addr ==
                                                        ifa->ifa_local) {
                                        break; /* found */
                                    sin_orig.sin_addr.s_addr ==
                                                        ifa->ifa_local) {
                                        break; /* found */
@@ -1044,7 +1037,7 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
                if (!ifa) {
                        for (ifap = &in_dev->ifa_list; (ifa = *ifap) != NULL;
                             ifap = &ifa->ifa_next)
                if (!ifa) {
                        for (ifap = &in_dev->ifa_list; (ifa = *ifap) != NULL;
                             ifap = &ifa->ifa_next)
-                               if (!strcmp(ifr.ifr_name, ifa->ifa_label))
+                               if (!strcmp(ifr->ifr_name, ifa->ifa_label))
                                        break;
                }
        }
                                        break;
                }
        }
@@ -1056,19 +1049,19 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
        switch (cmd) {
        case SIOCGIFADDR:       /* Get interface address */
                sin->sin_addr.s_addr = ifa->ifa_local;
        switch (cmd) {
        case SIOCGIFADDR:       /* Get interface address */
                sin->sin_addr.s_addr = ifa->ifa_local;
-               goto rarok;
+               break;
 
        case SIOCGIFBRDADDR:    /* Get the broadcast address */
                sin->sin_addr.s_addr = ifa->ifa_broadcast;
 
        case SIOCGIFBRDADDR:    /* Get the broadcast address */
                sin->sin_addr.s_addr = ifa->ifa_broadcast;
-               goto rarok;
+               break;
 
        case SIOCGIFDSTADDR:    /* Get the destination address */
                sin->sin_addr.s_addr = ifa->ifa_address;
 
        case SIOCGIFDSTADDR:    /* Get the destination address */
                sin->sin_addr.s_addr = ifa->ifa_address;
-               goto rarok;
+               break;
 
        case SIOCGIFNETMASK:    /* Get the netmask for the interface */
                sin->sin_addr.s_addr = ifa->ifa_mask;
 
        case SIOCGIFNETMASK:    /* Get the netmask for the interface */
                sin->sin_addr.s_addr = ifa->ifa_mask;
-               goto rarok;
+               break;
 
        case SIOCSIFFLAGS:
                if (colon) {
 
        case SIOCSIFFLAGS:
                if (colon) {
@@ -1076,11 +1069,11 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
                        if (!ifa)
                                break;
                        ret = 0;
                        if (!ifa)
                                break;
                        ret = 0;
-                       if (!(ifr.ifr_flags & IFF_UP))
+                       if (!(ifr->ifr_flags & IFF_UP))
                                inet_del_ifa(in_dev, ifap, 1);
                        break;
                }
                                inet_del_ifa(in_dev, ifap, 1);
                        break;
                }
-               ret = dev_change_flags(dev, ifr.ifr_flags);
+               ret = dev_change_flags(dev, ifr->ifr_flags);
                break;
 
        case SIOCSIFADDR:       /* Set interface address (and family) */
                break;
 
        case SIOCSIFADDR:       /* Set interface address (and family) */
@@ -1095,7 +1088,7 @@ int devinet_ioctl(struct net *net, unsigned int cmd, void __user *arg)
                                break;
                        INIT_HLIST_NODE(&ifa->hash);
                        if (colon)
                                break;
                        INIT_HLIST_NODE(&ifa->hash);
                        if (colon)
-                               memcpy(ifa->ifa_label, ifr.ifr_name, IFNAMSIZ);
+                               memcpy(ifa->ifa_label, ifr->ifr_name, IFNAMSIZ);
                        else
                                memcpy(ifa->ifa_label, dev->name, IFNAMSIZ);
                } else {
                        else
                                memcpy(ifa->ifa_label, dev->name, IFNAMSIZ);
                } else {
@@ -1182,10 +1175,6 @@ done:
        rtnl_unlock();
 out:
        return ret;
        rtnl_unlock();
 out:
        return ret;
-rarok:
-       rtnl_unlock();
-       ret = copy_to_user(arg, &ifr, sizeof(struct ifreq)) ? -EFAULT : 0;
-       goto out;
 }
 
 static int inet_gifconf(struct net_device *dev, char __user *buf, int len, int size)
 }
 
 static int inet_gifconf(struct net_device *dev, char __user *buf, int len, int size)