diff options
| author | Nathan Chancellor <natechancellor@gmail.com> | 2019-03-07 16:29:33 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2019-03-08 15:26:21 -0800 |
| commit | 09073525f8b9095931e5a82209b61abc7ce4c9d5 (patch) | |
| tree | 3639e38d35623ae7de651c4b861638a443da2580 /drivers/net/ethernet/sun/niu.c | |
| parent | 580411d07ce780c2dd721837ec1e42e30c5a58ee (diff) | |
| download | talos-op-linux-09073525f8b9095931e5a82209b61abc7ce4c9d5.tar.gz talos-op-linux-09073525f8b9095931e5a82209b61abc7ce4c9d5.zip | |
net: ethernet: sun: Zero initialize class in default case in niu_add_ethtool_tcam_entry
When building with -Wsometimes-uninitialized, Clang warns:
drivers/net/ethernet/sun/niu.c:7466:5: warning: variable 'class' is used
uninitialized whenever switch default is taken
[-Wsometimes-uninitialized]
The default case can never happen because i can only be 0 to 3
(NIU_L3_PROG_CLS is defined as 4). To make this clear to Clang,
just zero initialize class in the default case (use the macro
CLASS_CODE_UNRECOG to make it clear this shouldn't happen).
Link: https://github.com/ClangBuiltLinux/linux/issues/403
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/sun/niu.c')
| -rw-r--r-- | drivers/net/ethernet/sun/niu.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/sun/niu.c b/drivers/net/ethernet/sun/niu.c index d84501441edd..6f99437a6962 100644 --- a/drivers/net/ethernet/sun/niu.c +++ b/drivers/net/ethernet/sun/niu.c @@ -7464,6 +7464,7 @@ static int niu_add_ethtool_tcam_entry(struct niu *np, class = CLASS_CODE_USER_PROG4; break; default: + class = CLASS_CODE_UNRECOG; break; } ret = tcam_user_ip_class_set(np, class, 0, |
