Messages in this thread | ![/](/images/icornerl.gif) | | From | Arnd Bergmann <> | Date | Wed, 24 Apr 2019 20:45:46 +0200 | Subject | Re: [PATCH net-next] net: socket: Fix missing break in switch statement |
| |
On Wed, Apr 24, 2019 at 5:33 PM Gustavo A. R. Silva <gustavo@embeddedor.com> wrote: > > 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: 0768e17073dc ("net: socket: implement 64-bit timestamps") > Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Arnd Bergmann <arnd@arndb.de>
I had just prepared the same patch after Dan told me about the issue today, but you managed to submit it first.
Thanks for the fix!
Arnd
| ![\](/images/icornerr.gif) |