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)
commitb9908680126b3e5aa640de298c9382ea752b3be5
tree85c3b2f9420367dbc2d8669d974d64cb73ef4496
parent85a7a5e8e9740a4d5cd0aa4fefc16ae1f7fcc74d
parent9e6f2853625bee3b9dd670d6ad927e67cf176342
merged branch arnaud-lb/ext-crash2 (PR #560)

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.
ext/twig/twig.c