On Wed, Jan 21, 2004 at 01:02:03PM -0800, James Lamanna wrote: > > > > >The patch is trying to fix a real bug (the code never expects the number > >of usages to be less than the number of values in a report), however the > >fix is wrong, because later in the hid_add_field() function it can cause > >the loop whic assigns usages to the field->usage array to access data > >beyond the end of the parser->local.usage array. > > > >Could you do a proper fix? (Setting the field->usage array to the last > >defined usage?) > > > > If I'm understanding you correctly, I think the attached patch will prevent > reading beyond the parser usage array.
Yes, but we probably want to fill all the usages[] entries we allocated. -- Vojtech Pavlik SuSE Labs, SuSE CR ------------------------------------------------------- The SF.Net email is sponsored by EclipseCon 2004 Premiere Conference on Open Tools Development and Integration See the breadth of Eclipse activity. February 3-5 in Anaheim, CA. http://www.eclipsecon.org/osdn _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
