On Thu, Jul 05, 2012 at 04:37:39PM -0700, Keith Packard wrote: > Torsten Kaiser <[email protected]> writes: > > > With this optimization level gcc notices, that the loop in function > > DDCModesFromEstIII() would go until i=5 and j=1 which would result in > > m = (5 * 8) + (7 - 1) = 46, but the array EstIIIModes[] only contains > > 44 elements. > > I'd like Adam's opinion; he wrote the original code and is the best > person to verify that this does what the spec requires. On the face of > it, it looks right to me though...
I'm not a big fan of fixing two separate bugs in one patches though, I'd prefer to split out the out-of-bounds from the skipping last mode issue. Reviewed-by: Peter Hutterer <[email protected]> otherwise Cheers, Peter _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
