merged branch arnaud-lb/ext-crash2 (PR #560)
authorFabien Potencier <fabien.potencier@gmail.com>
Sat, 17 Dec 2011 07:58:24 +0000 (08:58 +0100)
committerFabien Potencier <fabien.potencier@gmail.com>
Sat, 17 Dec 2011 07:58:24 +0000 (08:58 +0100)
Commits
-------

9e6f285 fixed crash in twig_template_get_attributes when input is not an array or object

Discussion
----------

Fix crash in twig_template_get_attributes when input is not an array or object

This fixes a crash when the input of twig_template_get_attributes() is not an array or object, and strict mode is enabled.

---------------------------------------------------------------------------

by nikic at 2011/12/16 12:03:21 -0800

Could you maybe place the exception tests in a different method?

---------------------------------------------------------------------------

by arnaud-lb at 2011/12/16 12:13:35 -0800

This method already expects many of its inputs to cause an exception to be thrown (it tests getAttribute() in strict mode, and in many cases the attribute isn't defined). I don't see a real benefit from moving these tests in a separate method in this case.

1  2 
ext/twig/twig.c

diff --cc ext/twig/twig.c
Simple merge