Re: com php-src: Fixed bug #71148 (Bind reference overwritten on PHP 7): NEWS ext/oci8/oci8.c ext/oci8/oci8_interface.c ext/oci8/oci8_statement.c
ext/oci8/package.xml ext/oci8/php_oci8.h ext/oci8/php_oci8_int.h ext/oci8/tests/bug71148.phpt ext/oci8/tests/driver_name.phpt
| From: | Nikita Popov | Date: | Wed, 21 Dec 2016 21:19:30 +0000 |
| Subject: | Re: com php-src: Fixed bug #71148 (Bind reference overwritten on PHP 7): NEWS ext/oci8/oci8.c ext/oci8/oci8_interface.c ext/oci8/oci8_statement.c ext/oci8/package.xml ext/oci8/php_oci8.h ext/oci8/php_oci8_int.h ext/oci8/tests/bug71148.phpt ext/oci8/tests/driver_name.phpt |
||
| References: | 1 | Groups: | php.cvs |
| Request: | Send a blank email to php-cvs+get-96412@lists.php.net to get a copy of this message | ||
On Mon, Oct 17, 2016 at 3:40 AM, Christopher Jones <sixd@php.net> wrote:
> Commit: 8be59a13017f16396e862a8e50c888afafe1952d
> Author: Christopher Jones <christopher.jones@oracle.com> Mon,
> 17 Oct 2016 12:40:14 +1100
> Parents: d82da7de3a460f1eee3d34448be79bca3de7abc1
> Branches: PHP-7.0
>
> Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=
> 8be59a13017f16396e862a8e50c888afafe1952d
>
> Log:
> Fixed bug #71148 (Bind reference overwritten on PHP 7)
>
> Bugs:
> https://bugs.php.net/71148
>
> Changed paths:
> M NEWS
> M ext/oci8/oci8.c
> M ext/oci8/oci8_interface.c
> M ext/oci8/oci8_statement.c
> M ext/oci8/package.xml
> M ext/oci8/php_oci8.h
> M ext/oci8/php_oci8_int.h
> A ext/oci8/tests/bug71148.phpt
> M ext/oci8/tests/driver_name.phpt
>
There are a couple of places where bind->parameter is not initialized, so
you have an invalid memory read in the dtor. See
http://gcov.php.net/viewer.php?version=PHP_HEAD&func=valgrind&file=ext%2Foci8%2Ftests%2Farray_bind_003.phpt
for example.
Nikita