On Tue, May 11, 2010 at 11:20:02AM +0530, Sujith.Manoharan@xxxxxxxxxxx wrote: > Dan Carpenter wrote: > > The original code had ENDPOINT_MAX and HST_ENDPOINT_MAX switched. > > > > Also the first loop was off by one, it started past the end of the array > > and went down to 1 instead of going down to 0. The test at the end of > > the loop to see if we exited via a break wasn't right because > > "tmp_endpoint" is always non-null here. > > This is a very good catch and fixes a stack corruption issue. > Do you mind if I work upon this patch and send out an updated fix ? > > Sujith Sorry, I meant to do that yesterday but I was out of it. Yes. Please send the updated fix. regards, dan carpenter -- 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
