Commit 60747828 authored by Gustavo A. R. Silva's avatar Gustavo A. R. Silva Committed by David S. Miller

net: socket: Fix missing break in switch statement

Add missing break statement in order to prevent the code from falling
through to cases SIOCGSTAMP_NEW and SIOCGSTAMPNS_NEW.

This bug was found thanks to the ongoing efforts to enable
-Wimplicit-fallthrough.

Fixes: 0768e170 ("net: socket: implement 64-bit timestamps")
Signed-off-by: default avatarGustavo A. R. Silva <gustavo@embeddedor.com>
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarArnd Bergmann <arnd@arndb.de>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 0ff85d6d
...@@ -1173,6 +1173,7 @@ static long sock_ioctl(struct file *file, unsigned cmd, unsigned long arg) ...@@ -1173,6 +1173,7 @@ static long sock_ioctl(struct file *file, unsigned cmd, unsigned long arg)
err = sock->ops->gettstamp(sock, argp, err = sock->ops->gettstamp(sock, argp,
cmd == SIOCGSTAMP_OLD, cmd == SIOCGSTAMP_OLD,
!IS_ENABLED(CONFIG_64BIT)); !IS_ENABLED(CONFIG_64BIT));
break;
case SIOCGSTAMP_NEW: case SIOCGSTAMP_NEW:
case SIOCGSTAMPNS_NEW: case SIOCGSTAMPNS_NEW:
if (!sock->ops->gettstamp) { if (!sock->ops->gettstamp) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment