> From: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> > > Using is_zero_ether_addr() instead of directly use > memcmp() to determine if the ethernet address is all > zeros. > > spatch with a semantic match is used to found this problem. > (http://coccinelle.lip6.fr/) > > Signed-off-by: Wei Yongjun <yongjun_wei@xxxxxxxxxxxxxxxxx> Acked-by: Bing Zhao <bzhao@xxxxxxxxxxx> Thanks, Bing > --- > drivers/net/wireless/mwifiex/sta_cmdresp.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/mwifiex/sta_cmdresp.c b/drivers/net/wireless/mwifiex/sta_cmdresp.c > index 0b09004..62223ab 100644 > --- a/drivers/net/wireless/mwifiex/sta_cmdresp.c > +++ b/drivers/net/wireless/mwifiex/sta_cmdresp.c > @@ -736,7 +736,6 @@ static int mwifiex_ret_ibss_coalescing_status(struct mwifiex_private *priv, > { > struct host_cmd_ds_802_11_ibss_status *ibss_coal_resp = > &(resp->params.ibss_coalescing); > - u8 zero_mac[ETH_ALEN] = { 0, 0, 0, 0, 0, 0 }; > > if (le16_to_cpu(ibss_coal_resp->action) == HostCmd_ACT_GEN_SET) > return 0; > @@ -745,7 +744,7 @@ static int mwifiex_ret_ibss_coalescing_status(struct mwifiex_private *priv, > "info: new BSSID %pM\n", ibss_coal_resp->bssid); > > /* If rsp has NULL BSSID, Just return..... No Action */ > - if (!memcmp(ibss_coal_resp->bssid, zero_mac, ETH_ALEN)) { > + if (is_zero_ether_addr(ibss_coal_resp->bssid)) { > dev_warn(priv->adapter->dev, "new BSSID is NULL\n"); > return 0; > } > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
