On 01/10/2012 06:23 AM, Sujith Manoharan wrote: > This prevents 'comp_pktq' from being used in an > incorrect manner. > > Signed-off-by: Sujith Manoharan <c_manoha@xxxxxxxxxxxxxxxx> > --- > drivers/net/wireless/ath/ath6kl/htc.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/drivers/net/wireless/ath/ath6kl/htc.c b/drivers/net/wireless/ath/ath6kl/htc.c > index b017022..2d72190 100644 > --- a/drivers/net/wireless/ath/ath6kl/htc.c > +++ b/drivers/net/wireless/ath/ath6kl/htc.c > @@ -2062,6 +2062,7 @@ int ath6kl_htc_rxmsg_pending_handler(struct htc_target *target, > enum htc_endpoint_id id; > int n_fetched = 0; > > + INIT_LIST_HEAD(&comp_pktq); But there's also the same init later in the function, inside the while loop: INIT_LIST_HEAD(&rx_pktq); INIT_LIST_HEAD(&comp_pktq); Is this intentional? Kalle -- 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