[Bug 102685] piglit.spec.glsl-1_50.compiler.vs-redeclares-pervertex-out-before-global-redeclaration

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=102685

--- Comment #1 from Iago Toral <itoral at igalia.com> ---
Hi Mark, sorry about this, I am not sure why I didn't see this before,  I was
sure that I had sent this patch to CI before sending it for review :-/

Anyway, the problem is not with piglit, there is some rather unsafe code right
prior to the place where this commit operated that can invalidate the memory of
'var' in one specific scenario (triggered by these parser tests) that makes any
access to that variable from that point forward potentially unsafe: the
variable ends up pointing to freed memory so you get undefined behavior, so the
crash is not even guaranteed to happen. I think this is quite unfortunate, so
I'll be sending a couple of patches, one to fix this regression and one or two
more to make that implementation safer so we don't see more instances of this
type of problems in the future.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/intel-3d-bugs/attachments/20170913/1fb9213a/attachment-0001.html>