Skip to content

Commit

Permalink
Fix Bug #75299 Wrong reflection on inflate_init and inflate_add
Browse files Browse the repository at this point in the history
  • Loading branch information
villfa committed Oct 1, 2017
1 parent abefb6d commit 9d9873b
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 2 deletions.
20 changes: 20 additions & 0 deletions ext/zlib/tests/bug75299.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
--TEST--
Bug #75299 Wrong reflection on inflate_init and inflate_add
--SKIPIF--
<?php if (!extension_loaded("zlib") || !extension_loaded('reflection')) die("skip"); ?>
--FILE--
<?php
$r = new ReflectionFunction('inflate_init');
var_dump($r->getNumberOfRequiredParameters());
var_dump($r->getNumberOfParameters());
$r = new ReflectionFunction('inflate_add');
var_dump($r->getNumberOfRequiredParameters());
var_dump($r->getNumberOfParameters());
?>
===DONE===
--EXPECT--
int(1)
int(2)
int(2)
int(3)
===DONE===
6 changes: 4 additions & 2 deletions ext/zlib/zlib.c
Original file line number Diff line number Diff line change
Expand Up @@ -1315,11 +1315,13 @@ ZEND_END_ARG_INFO()

ZEND_BEGIN_ARG_INFO_EX(arginfo_inflate_init, 0, 0, 1)
ZEND_ARG_INFO(0, encoding)
ZEND_ARG_INFO(0, options)
ZEND_END_ARG_INFO()

ZEND_BEGIN_ARG_INFO_EX(arginfo_inflate_add, 0, 0, 2)
ZEND_ARG_INFO(0, resource)
ZEND_ARG_INFO(0, flush_behavior)
ZEND_ARG_INFO(0, context)
ZEND_ARG_INFO(0, encoded_data)
ZEND_ARG_INFO(0, flush_mode)
ZEND_END_ARG_INFO()

/* }}} */
Expand Down

0 comments on commit 9d9873b

Please sign in to comment.