On Tue, May 11, 2010 at 04:25:32PM +0530, Sujith.Manoharan@xxxxxxxxxxx wrote: > Use ENDPOINT_MAX instead of HST_ENDPOINT_MAX. > This fixes a stack corruption issue. > > This is based on a patch sent by Dan Carpenter <error27@xxxxxxxxx>. > There is a bit missing. The tmp_endpoint variable is always non-null here. Can you just roll this into your patch? Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> diff --git a/drivers/net/wireless/ath/ath9k/htc_hst.c b/drivers/net/wireless/ath/ath9k/htc_hst.c index 6a062a3..02e8e0f 100644 --- a/drivers/net/wireless/ath/ath9k/htc_hst.c +++ b/drivers/net/wireless/ath/ath9k/htc_hst.c @@ -124,7 +124,7 @@ static void htc_process_conn_rsp(struct htc_target *target, } } - if (!tmp_endpoint) + if (tepid == ENDPOINT0) return; endpoint->service_id = service_id; -- 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
