On Tue, Mar 8, 2016 at 9:54 AM, Iago Toral <ito...@igalia.com> wrote: > On Sun, 2016-03-06 at 12:21 -0500, Ilia Mirkin wrote: >> This fixes a crash in >> >> dEQP-GLES3.functional.transform_feedback.array_element.separate.points.lowp_mat3x2 >> >> and likely others. The vertex shader has > 16 input variables (without >> explicit locations), which causes us to index outside of the to_assign >> array. >> >> Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu> >> Cc: "11.1 11.2" <mesa-sta...@lists.freedesktop.org> >> --- >> >> Not sure if this is the right thing or whether we should be making the >> to_assign array dynamically resizable. This definitely fixes the crashes >> though. > > FWIW, I had sent a similar patch about a year ago to fix this: > https://lists.freedesktop.org/archives/mesa-dev/2015-July/090121.html > > Not sure that it is any better...
Maybe a little? Sorry, didn't realize that a fix for this already existed... using max_index is probably a little bit better? I've already pushed my version, but feel free to do yours as a fixup. If you guys have additional useful patches that have remained unreviewed, it'd probably be good to ping people to review them every so often. These things are quickly forgotten, and if the author(s) don't push for inclusion, it will be lost forever. -ilia _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
