linux-kernel - Re: linux-next: build warnings after merge of the cifs tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAH2r5mvtFzp=nYK+J92vh9y4tsNLT3rD4a2+mB9LMyWcFN-EHA@mail.gmail.com>
Date:   Sun, 25 Nov 2018 18:31:40 -0600
From:   Steve French <smfrench@...il.com>
To:     Stephen Rothwell <sfr@...b.auug.org.au>,
        Sachin Prabhu <sprabhu@...hat.com>
Cc:     CIFS <linux-cifs@...r.kernel.org>, keescook@...gle.com,
        Linux-Next Mailing List <linux-next@...r.kernel.org>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: linux-next: build warnings after merge of the cifs tree

Both of those cases are intentional fallthroughs and there are
existing comments in the code noting the reasons for them to
fallthrough

(also can see the reasoning for these in the commits which introduced
them from Sachin c369c9a4a7c82) and dde2356c84662)
On Sun, Nov 25, 2018 at 6:14 PM Stephen Rothwell <sfr@...b.auug.org.au> wrote:
>
> Hi all,
>
> After merging the cifs tree, today's linux-next build (powerpc
> ppc64_defconfig) produced these warnings:
>
> fs/cifs/sess.c: In function 'cifs_select_sectype':
> fs/cifs/sess.c:527:3: warning: this statement may fall through [-Wimplicit-fallthrough=]
>    switch (requested) {
>    ^~~~~~
> fs/cifs/sess.c:540:2: note: here
>   case CIFS_NEGFLAVOR_LANMAN:
>   ^~~~
> fs/cifs/connect.c: In function 'cifs_parse_mount_options':
> fs/cifs/connect.c:1825:8: warning: this statement may fall through [-Wimplicit-fallthrough=]
>     if (!(tmp_end < end && tmp_end[1] == delim)) {
>         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> fs/cifs/connect.c:1832:3: note: here
>    case Opt_pass:
>    ^~~~
>
> Exposed by my use of -Wimplicit-fallthrough
>
> I am not sure why these only turned up now (as opposed to earlier today).
>
> --
> Cheers,
> Stephen Rothwell



-- 
Thanks,

Steve