crash in ConstantUnion::getBConst with this=null, called from TIntermediate::addSelection, from yyparse

[GoogleCodeExporter]

This is https://bugzilla.mozilla.org/show_bug.cgi?id=620222

Testcase: https://bugzilla.mozilla.org/attachment.cgi?id=498607

Shader pattern:

  const highp bvec2 cac;
  void main() { if((cac = 4294967295))for(; ; ){} }

Call stack: https://bugzilla.mozilla.org/attachment.cgi?id=498596

#0  0x0000000102dfe949 in ConstantUnion::getBConst (this=0x0) at 
ConstantUnion.h:22
#1  0x0000000102df62e1 in TIntermediate::addSelection (this=0x7fff5fbfbe10, 
cond=0x127621628, nodePair={node1 = 0x127621e78, node2 = 0x0}, line=2) at 
/Users/cdiehl/Mozilla/trunk/gfx/angle/src/compiler/Intermediate.cpp:595
#2  0x0000000102e4329c in yyparse (context=0x7fff5fbfbd00) at 
/Users/cdiehl/Mozilla/trunk/gfx/angle/src/compiler/glslang_tab.cpp:4158
#3  0x0000000102e4533c in glslang_parse (context=0x7fff5fbfbd00) at 
/Users/cdiehl/Mozilla/trunk/gfx/angle/src/compiler/glslang_tab.cpp:4707
#4  0x0000000102e0d166 in PaParseStrings (count=1, string=0x7fff5fbfbf30, 
length=0x0, context=0x7fff5fbfbd00) at 
/Users/cdiehl/Mozilla/trunk/gfx/angle/src/compiler/ParseHelper.cpp:1436
#5  0x0000000102de0e9d in TCompiler::compile (this=0x12df4ed60, 
shaderStrings=0x7fff5fbfbf30, numStrings=1, compileOptions=5) at 
/Users/cdiehl/Mozilla/trunk/gfx/angle/src/compiler/Compiler.cpp:132
#6  0x0000000102e1b23e in ShCompile (handle=0x12df4ed60, 
shaderStrings=0x7fff5fbfbf30, numStrings=1, compileOptions=4) at 
/Users/cdiehl/Mozilla/trunk/gfx/angle/src/compiler/ShaderLang.cpp:164

Original issue reported on code.google.com by jacob.be...@gmail.com on 2 Nov 2011 at 8:29

• Merged into: #381

[GoogleCodeExporter]

Original comment by jacob.be...@gmail.com on 2 Nov 2011 at 8:34

• Changed title: crash in ConstantUnion::getBConst with this=null, called from TIntermediate::addSelection, from yyparse

[GoogleCodeExporter]

Original comment by kbr@chromium.org on 4 Nov 2011 at 12:44

[GoogleCodeExporter]

Original comment by z...@google.com on 27 Feb 2013 at 9:26

• Changed state: Duplicate