On Tue, Mar 04, 2014 at 05:07:23PM +0100, Arvid Brodin wrote: > On 2014-03-04 04:27, Dave Jones wrote: > > I found this in coverity, and I think it's a real bug.. > > > > hsr_register_frame_in does a check that dev_idx is between 0 and 2, > > therefore, a dev_idx of 2 is possible when it gets to the array writes > > at the end of the function. > > Thanks for finding this; it is a bug (although I don't think it has > actually lead to any out of bound accesses). > > However, I think you are a bit late - I believe this was fixed in a patch > from Dan Carpenter just a few days ago. See > > /lists/netdev/msg272815.html excellent, thanks for checking. Dave -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html