l2tp: fix set but not used variable
authorJakub Kicinski <jakub.kicinski@netronome.com>
Wed, 17 Apr 2019 20:51:55 +0000 (13:51 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 19 Apr 2019 00:06:15 +0000 (17:06 -0700)
GCC complains:

net/l2tp/l2tp_ppp.c: In function ‘pppol2tp_ioctl’:
net/l2tp/l2tp_ppp.c:1073:6: warning: variable ‘val’ set but not used [-Wunused-but-set-variable]
  int val;
      ^~~

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Dirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/l2tp/l2tp_ppp.c

index 04d9946dcdba648f8e4dce9c0c0211f7036ae135..f36cae785e8217be3e9aed72d5fe7c01939511b6 100644 (file)
@@ -1070,7 +1070,6 @@ static int pppol2tp_ioctl(struct socket *sock, unsigned int cmd,
 {
        struct pppol2tp_ioc_stats stats;
        struct l2tp_session *session;
-       int val;
 
        switch (cmd) {
        case PPPIOCGMRU:
@@ -1097,7 +1096,7 @@ static int pppol2tp_ioctl(struct socket *sock, unsigned int cmd,
                if (!session->session_id && !session->peer_session_id)
                        return -ENOSYS;
 
-               if (get_user(val, (int __user *)arg))
+               if (!access_ok((int __user *)arg, sizeof(int)))
                        return -EFAULT;
                break;