Hi Dan, Thanks for the patch. > -----Original Message----- > From: Dan Carpenter [mailto:dan.carpenter@xxxxxxxxxx] > Sent: Wednesday, September 21, 2011 12:14 AM > To: Bing Zhao > Cc: John W. Linville; linux-wireless@xxxxxxxxxxxxxxx; kernel-janitors@xxxxxxxxxxxxxxx > Subject: [patch 2/2] mwifiex: remove unneeded NULL check > > We dereference "rate" on the lines before so the checks here are too > late to help. This function is only called from > mwifiex_dump_station_info() and "rate" is always a non-NULL pointer > so the check can be removed. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: Bing Zhao <bzhao@xxxxxxxxxxx> Thanks, Bing > > diff --git a/drivers/net/wireless/mwifiex/sta_ioctl.c b/drivers/net/wireless/mwifiex/sta_ioctl.c > index 215c65a..1df5ef6 100644 > --- a/drivers/net/wireless/mwifiex/sta_ioctl.c > +++ b/drivers/net/wireless/mwifiex/sta_ioctl.c > @@ -868,10 +868,10 @@ int mwifiex_drv_get_data_rate(struct mwifiex_private *priv, > ret = mwifiex_rate_ioctl_cfg(priv, rate); > > if (!ret) { > - if (rate && rate->is_rate_auto) > + if (rate->is_rate_auto) > rate->rate = mwifiex_index_to_data_rate(priv->tx_rate, > priv->tx_htinfo); > - else if (rate) > + else > rate->rate = priv->data_rate; > } else { > ret = -1; -- 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
