Fix bug #71540 - NULL pointer dereference in xsl_ext_function_php()

This commit is contained in:
Stanislav Malyshev 2016-02-14 23:35:29 -08:00
parent e18910c296
commit 889e3b62f4
2 changed files with 71 additions and 0 deletions

View file

@ -0,0 +1,67 @@
--TEST--
Bug #71540 (NULL pointer dereference in xsl_ext_function_php())
--SKIPIF--
<?php
if (!extension_loaded('xsl')) die("skip Extension XSL is required\n");
?>
--FILE--
<?php
$xml = <<<EOB
<allusers>
<user>
<uid>bob</uid>
</user>
</allusers>
EOB;
$xsl = <<<EOB
<?xml version="1.0" encoding="UTF-8"?>
<xsl:stylesheet version="1.0"
xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
xmlns:php="http://php.net/xsl">
<xsl:output method="html" encoding="utf-8" indent="yes"/>
<xsl:template match="allusers">
<html><body>
<h2>Users</h2>
<table>
<xsl:for-each select="user">
<tr><td>
<xsl:value-of
select="php:function('test',uid,test(test))"/>
</td></tr>
</xsl:for-each>
</table>
</body></html>
</xsl:template>
</xsl:stylesheet>
EOB;
$xmldoc = new DOMDocument();
$xmldoc->loadXML($xml);
$xsldoc = new DOMDocument();
$xsldoc->loadXML($xsl);
$proc = new XSLTProcessor();
$proc->registerPHPFunctions();
$proc->importStyleSheet($xsldoc);
echo $proc->transformToXML($xmldoc);
?>
DONE
--EXPECTF--
Warning: XSLTProcessor::transformToXml(): xmlXPathCompOpEval: function test not found in %sbug71540.php on line %d
Warning: XSLTProcessor::transformToXml(): Unregistered function in %sbug71540.php on line %d
Warning: XSLTProcessor::transformToXml(): Stack usage errror in %sbug71540.php on line %d
Warning: XSLTProcessor::transformToXml(): Stack usage errror in %sbug71540.php on line %d
Warning: XSLTProcessor::transformToXml(): xmlXPathCompiledEval: 2 objects left on the stack. in %sbug71540.php on line %d
Warning: XSLTProcessor::transformToXml(): runtime error: file %s line 13 element value-of in %sbug71540.php on line %d
Warning: XSLTProcessor::transformToXml(): XPath evaluation returned no result. in %sbug71540.php on line %d
<html xmlns:php="http://php.net/xsl"><body>
<h2>Users</h2>
<table><tr><td></td></tr></table>
</body></html>
DONE

View file

@ -231,6 +231,10 @@ static void xsl_ext_function_php(xmlXPathParserContextPtr ctxt, int nargs, int t
/* Reverse order to pop values off ctxt stack */
for (i = nargs - 2; i >= 0; i--) {
obj = valuePop(ctxt);
if (obj == NULL) {
ZVAL_NULL(&args[i]);
continue;
}
switch (obj->type) {
case XPATH_STRING:
ZVAL_STRING(&args[i], (char *)obj->stringval);