linux-kernel - Re: [PATCH 00/20] mark expected switch fall-throughs

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] [day] [month] [year] [list]

Message-ID: <20171023143337.Horde.Bm233ao94-f7wfQrmA8JqeS@gator4166.hostgator.com>
Date:   Mon, 23 Oct 2017 14:33:37 -0500
From:   "Gustavo A. R. Silva" <garsilva@...eddedor.com>
To:     linux-rpi-kernel@...ts.infradead.org,
        linux-arm-kernel@...ts.infradead.org,
        lustre-devel@...ts.lustre.org, devel@...verdev.osuosl.org,
        linux-kernel@...r.kernel.org
Cc:     Oleg Drokin <oleg.drokin@...el.com>,
        Andreas Dilger <andreas.dilger@...el.com>,
        James Simmons <jsimmons@...radead.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Ian Abbott <abbotti@....co.uk>,
        H Hartley Sweeten <hsweeten@...ionengravers.com>,
        Eric Anholt <eric@...olt.net>,
        Stefan Wahren <stefan.wahren@...e.com>,
        Forest Bond <forest@...ttletooquiet.net>
Subject: Re: [PATCH 00/20] mark expected switch fall-throughs


Quoting "Gustavo A. R. Silva" <garsilva@...eddedor.com>:

> In preparation to enabling -Wimplicit-fallthrough, this patchset aims
> to mark switch cases where we are expecting to fall through.
>
> In Kees Cook words:
> "This is an unfortunate omission in the C language, and thankfully both
> gcc and clang have stepped up to solve this the same way static
> analyzers have solved it. It does both document the intention for
> humans and provide a way for analyzers to report issues.
>
> Having the compiler help us not make mistakes is quite handy."
>
> In some cases there were "no break" or "fall thru" comments already
> in place. So I replaced them with proper "fall through" comments, which
> is what GCC is expecting to find.
>
> For the rest of the cases, please double check if the actual intention
> of the code is to fall through.
>
> Thanks!
>
> Gustavo A. R. Silva (20):
>   staging: ks7010: ks_wlan_net: mark expected switch fall-throughs
>   staging: lustre: lnet: socklnd: mark expected switch fall-through
>   staging: rtl8192e: mark expected switch fall-through
>   staging: rtl8723bs: rtw_mlme_ext: mark expected switch fall-through
>   staging: lustre: lnet: net_fault: mark expected switch fall-through
>   staging: comedi: s526: mark expected switch fall-through
>   staging: rtl8188eu: usb_halinit: mark expected switch fall-through
>   staging: vc04_services: vchiq_core: mark expected switch fall-through
>   staging: vt6656: card: mark expected switch fall-throughs
>   staging: rtl8188eu: usb_ops_linux: mark expected switch fall-through
>   staging: r8822be: mark expected switch fall-throughs
>   staging: lustre: lnet: selftest: mark expected switch fall-through
>   staging: rtlwifi: halmac: mark expected switch fall-through
>   staging: lustre: lnet: selftest: mark expected switch fall-throughs
>   staging: lustre: llite: mark expected switch fall-through
>   staging: lustre: lprocfs: mark expected switch fall-throughs
>   staging: lustre: ldlm: mark expected switch fall-through
>   staging: lustre: osc: mark expected switch fall-through
>   staging: lustre: ptlrpc: mark expected switch fall-throughs
>   staging: lustre: rpc: mark expected switch fall-throughs
>
>  drivers/staging/comedi/drivers/s526.c                       |  5 ++---
>  drivers/staging/ks7010/ks_wlan_net.c                        | 10 ++++++++++
>  drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c         |  2 +-
>  drivers/staging/lustre/lnet/lnet/net_fault.c                |  1 +
>  drivers/staging/lustre/lnet/selftest/conctl.c               |  1 +
>  drivers/staging/lustre/lnet/selftest/module.c               |  5 ++++-
>  drivers/staging/lustre/lnet/selftest/rpc.c                  | 13  
> +++++++++----
>  drivers/staging/lustre/lustre/ldlm/ldlm_request.c           |  1 +
>  drivers/staging/lustre/lustre/llite/namei.c                 |  4 +++-
>  drivers/staging/lustre/lustre/obdclass/lprocfs_status.c     |  4 ++++
>  drivers/staging/lustre/lustre/osc/osc_cache.c               |  1 +
>  drivers/staging/lustre/lustre/ptlrpc/pack_generic.c         |  6 +++---
>  drivers/staging/rtl8188eu/hal/usb_halinit.c                 |  1 +
>  drivers/staging/rtl8188eu/os_dep/usb_ops_linux.c            |  1 +
>  drivers/staging/rtl8192e/rtllib_wx.c                        |  3 +--
>  drivers/staging/rtl8723bs/core/rtw_mlme_ext.c               |  2 +-
>  .../staging/rtlwifi/halmac/halmac_88xx/halmac_api_88xx.c    |  2 ++
>  drivers/staging/rtlwifi/halmac/rtl_halmac.c                 |  2 +-
>  .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c  |  3 +--
>  drivers/staging/vt6656/card.c                               |  2 ++
>  20 files changed, 50 insertions(+), 19 deletions(-)
>
> --
> 2.7.4

Andreas, Stefan:
Thank you for your reviews and ACKs

Greg:
Thank you for applying the patches.

--
Gustavo A. R. Silva