Re: [PATCH] net: dsa: mv88e6xxx: add default case to switch

From	David Miller <davem@davemloft.net>
Newsgroups	linux.kernel
Subject	Re: [PATCH] net: dsa: mv88e6xxx: add default case to switch
Date	2017-05-12 18:20 +0200
Message-ID	<tGlsm-76m-25@gated-at.bofh.it> (permalink)
References	<tG8Ot-6rP-3@gated-at.bofh.it> <tG9hw-6RZ-11@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

From: "Gustavo A. R. Silva" <garsilva@embeddedor.com>
Date: Thu, 11 May 2017 22:11:29 -0500

> Add default case to switch in order to avoid any chance of using an
> uninitialized variable _low_, in case s->type does not match any of
> the listed case values.
> 
> Addresses-Coverity-ID: 1398130
> Suggested-by: Andrew Lunn <andrew@lunn.ch>
> Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>

Applied, thanks.

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[net-dsa-mv88e6xxx] question about potential use of uninitialized  variable "Gustavo A. R. Silva" <garsilva@embeddedor.com> - 2017-05-11 23:40 +0200
  Re: [net-dsa-mv88e6xxx] question about potential use of  uninitialized variable Andrew Lunn <andrew@lunn.ch> - 2017-05-12 04:40 +0200
    Re: [net-dsa-mv88e6xxx] question about potential use of  uninitialized variable "Gustavo A. R. Silva" <garsilva@embeddedor.com> - 2017-05-12 04:50 +0200
      [PATCH] net: dsa: mv88e6xxx: add default case to switch "Gustavo A. R. Silva" <garsilva@embeddedor.com> - 2017-05-12 05:20 +0200
        Re: [PATCH] net: dsa: mv88e6xxx: add default case to switch David Miller <davem@davemloft.net> - 2017-05-12 18:20 +0200

csiph-web