ethtool: Remove unnecessary null check in ethtool_rx_flow_rule_create
authorNathan Chancellor <natechancellor@gmail.com>
Fri, 8 Feb 2019 04:46:53 +0000 (21:46 -0700)
committerDavid S. Miller <davem@davemloft.net>
Sat, 9 Feb 2019 07:05:18 +0000 (23:05 -0800)
net/core/ethtool.c:3023:19: warning: address of array
'ext_m_spec->h_dest' will always evaluate to 'true'
[-Wpointer-bool-conversion]
                if (ext_m_spec->h_dest) {
                ~~  ~~~~~~~~~~~~^~~~~~

h_dest is an array, it can't be null so remove this check.

Fixes: eca4205f9ec3 ("ethtool: add ethtool_rx_flow_spec to flow_rule structure translator")
Link: https://github.com/ClangBuiltLinux/linux/issues/353
Signed-off-by: Nathan Chancellor <natechancellor@gmail.com>
Acked-by: Pablo Neira Ayuso <pablo@netfilter.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/ethtool.c

index 0fbf39239b29147431f7d8899d284dc30d6ee6ef..d2c47cdf25da743e28315c3f28144258ab1c9ef9 100644 (file)
@@ -3020,17 +3020,15 @@ ethtool_rx_flow_rule_create(const struct ethtool_rx_flow_spec_input *input)
                const struct ethtool_flow_ext *ext_h_spec = &fs->h_ext;
                const struct ethtool_flow_ext *ext_m_spec = &fs->m_ext;
 
-               if (ext_m_spec->h_dest) {
-                       memcpy(match->key.eth_addrs.dst, ext_h_spec->h_dest,
-                              ETH_ALEN);
-                       memcpy(match->mask.eth_addrs.dst, ext_m_spec->h_dest,
-                              ETH_ALEN);
-
-                       match->dissector.used_keys |=
-                               BIT(FLOW_DISSECTOR_KEY_ETH_ADDRS);
-                       match->dissector.offset[FLOW_DISSECTOR_KEY_ETH_ADDRS] =
-                               offsetof(struct ethtool_rx_flow_key, eth_addrs);
-               }
+               memcpy(match->key.eth_addrs.dst, ext_h_spec->h_dest,
+                      ETH_ALEN);
+               memcpy(match->mask.eth_addrs.dst, ext_m_spec->h_dest,
+                      ETH_ALEN);
+
+               match->dissector.used_keys |=
+                       BIT(FLOW_DISSECTOR_KEY_ETH_ADDRS);
+               match->dissector.offset[FLOW_DISSECTOR_KEY_ETH_ADDRS] =
+                       offsetof(struct ethtool_rx_flow_key, eth_addrs);
        }
 
        act = &flow->rule->action.entries[0];