http://code.google.com/p/angleproject/issues/detail?id=360 When RenameFunction::RenameFunction(const TString& oldFunctionName, const TString& newFunctionName) is called from TCompiler::rewriteCSSShader(TIntermNode*), references to the temporaries oldFunctionName and newFunctionName are stored as mOldFunctionName and mNewFunctionName. This results in undefined behaviour in visitAggregate. Possible fix in RenameFunction.h: private: - const TString& mOldFunctionName; + const TString mOldFunctionName; - const TString& mNewFunctionName; + const TString mNewFunctionName;
Good catch. Thanks Joshua. We'll fix this in ANGLE first.
Created attachment 160263 [details] Patch
Thanks for the patch! FYI, you don't need to worry about the style bot when you're touching ANGLE code. We have bug 90909 to teach the style bot to ignore the ANGLE directory.
(In reply to comment #1) > Good catch. Thanks Joshua. We'll fix this in ANGLE first. Are you saying you would rather the patch not land here but instead in ANGLE first?
The whole ANGLE directory gets replaced with the latest version from Google every once in a while. IMO, there's no point in updating all of ANGLE just for this.
(In reply to comment #4) > (In reply to comment #1) > > Good catch. Thanks Joshua. We'll fix this in ANGLE first. > > Are you saying you would rather the patch not land here but instead in ANGLE first? That's what I was saying, but I think it's fine to land this now, since we have a patch prepped for ANGLE already. (In reply to comment #5) > The whole ANGLE directory gets replaced with the latest version from Google every once in a while. IMO, there's no point in updating all of ANGLE just for this. I totally agree.
Comment on attachment 160263 [details] Patch Clearing flags on attachment: 160263 Committed r126625: <http://trac.webkit.org/changeset/126625>
All reviewed patches have been landed. Closing bug.
Fix has been upstreamed in ANGLE r1266: http://code.google.com/p/angleproject/source/detail?r=1266