tony2001Mon May 29 10:23:53 2006 UTC
Modified files:
/php-src/ext/standard file.c
Log:
prevent resource leaking when passing invalid stream resource as second
argument
http://cvs.php.net/viewcvs.cgi/php-src/ext/standard/file.c?r1=1.442&r2=1.443
tony2001Mon May 29 10:24:49 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/standard file.c
Log:
MFH: prevent resource leaking when passing invalid stream resource as second
argument
http://cvs.php.net/viewcvs.cgi/php-src/ext/standard/fi
tony2001Mon May 29 10:39:59 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/standard file.c
Log:
initialize srcstream
http://cvs.php.net/viewcvs.cgi/php-src/ext/standard/file.c?r1=1.409.2.6.2.2&r2=1.409.2.6.2.3&diff_format=u
Index: php-sr
tony2001Mon May 29 10:42:10 2006 UTC
Modified files:
/php-src/ext/standard file.c
Log:
initialize srcstream
use different variable in local scope
http://cvs.php.net/viewcvs.cgi/php-src/ext/standard/file.c?r1=1.443&r2=1.444&diff_format=u
Index:
tony2001Mon May 29 10:42:37 2006 UTC
Added files:
/php-src/ext/standard/tests/filefile_put_contents.phpt
Log:
add test
http://cvs.php.net/viewcvs.cgi/php-src/ext/standard/tests/file/file_put_contents.phpt?view=markup&rev=1.1
Index: php-src/ext
tony2001Mon May 29 10:42:54 2006 UTC
Added files: (Branch: PHP_5_2)
/php-src/ext/standard/tests/filefile_put_contents.phpt
Log:
add test
http://cvs.php.net/viewcvs.cgi/php-src/ext/standard/tests/file/file_put_contents.phpt?view=markup&rev=1.1
I
Could someone fix this damned CVS log script ?
Currently it shows data from HEAD when a different data has been actually
committed to a branch, which is kinda confusing.
And those infamous "empty log mails" when somebody adds a file only to a
branch..
On 29.05.2006 14:42, Antony Dovgal wrote:
tony2001Mon May 29 10:59:15 2006 UTC
Modified files:
/php-src/ext/oci8 oci8.c
Log:
fix #37531 (oci8 persistent connection corruption)
http://cvs.php.net/viewcvs.cgi/php-src/ext/oci8/oci8.c?r1=1.299&r2=1.300&diff_format=u
Index: php-src/ext/oci8/oci8
tony2001Mon May 29 10:59:30 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/oci8 oci8.c
Log:
MFH: fix #37531 (oci8 persistent connection corruption)
http://cvs.php.net/viewcvs.cgi/php-src/ext/oci8/oci8.c?r1=1.269.2.16.2.2&r2=1.269.2.16.2.3
tony2001Mon May 29 11:00:33 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-srcNEWS
Log:
BFN
http://cvs.php.net/viewcvs.cgi/php-src/NEWS?r1=1.2027.2.547.2.60&r2=1.2027.2.547.2.61&diff_format=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.547.2
mikeMon May 29 16:53:56 2006 UTC
Added files: (Branch: PHP_5_2)
/php-src/ext/mysqli/tests bug36949.phpt
Modified files:
/php-srcNEWS
/php-src/ext/mysqli mysqli.c
Log:
- Fixed bug #36949 (invalid internal mysqli objects dtor)
mikeMon May 29 16:55:40 2006 UTC
Modified files:
/php-src/ext/mysqli mysqli.c
/php-src/ext/mysqli/tests bug36949.phpt
Log:
MFB: Fixed bug #36949 (invalid internal mysqli objects dtor)
http://cvs.php.net/viewcvs.cgi/php-src/ext/mysqli/mysqli.c?r1=1.
tony2001Mon May 29 18:12:39 2006 UTC
Modified files:
/php-src/ext/oci8 config.m4
Log:
improve sed expressions
http://cvs.php.net/viewcvs.cgi/php-src/ext/oci8/config.m4?r1=1.59&r2=1.60&diff_format=u
Index: php-src/ext/oci8/config.m4
diff -u php-src/e
tony2001Mon May 29 18:12:55 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/oci8 config.m4
Log:
MFH: improve sed expressions
http://cvs.php.net/viewcvs.cgi/php-src/ext/oci8/config.m4?r1=1.55.2.3&r2=1.55.2.3.2.1&diff_format=u
Index: php-src/ex
Hello Antony,
i have a different set of scripts based on Ken Coar's that work perfectly
at my office. But Rasmus had problems setting them up. So i am not sure it
is a good idea to install them and try a bit until the issues are solved.
For example we should block access to all but a test reposi
helly Mon May 29 20:07:40 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-srcNEWS
Log:
- BFN
http://cvs.php.net/viewcvs.cgi/php-src/NEWS?r1=1.2027.2.547.2.62&r2=1.2027.2.547.2.63&diff_format=u
Index: php-src/NEWS
diff -u php-src/NEWS:1.2027.2.547.2.62 php-s
tony2001Mon May 29 20:26:12 2006 UTC
Modified files:
/php-src/ext/pcre php_pcre.c
Log:
return NULL in case of error (and fix leak/non-null-terminated string error)
http://cvs.php.net/viewcvs.cgi/php-src/ext/pcre/php_pcre.c?r1=1.181&r2=1.182&diff_for
tony2001Mon May 29 20:26:32 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/pcre php_pcre.c
Log:
MFH: return NULL in case of error
http://cvs.php.net/viewcvs.cgi/php-src/ext/pcre/php_pcre.c?r1=1.168.2.9.2.2&r2=1.168.2.9.2.3&diff_format=u
Inde
tony2001Mon May 29 20:31:33 2006 UTC
Added files:
/php-src/ext/pcre/tests invalid_utf8.phpt
Log:
add test
http://cvs.php.net/viewcvs.cgi/php-src/ext/pcre/tests/invalid_utf8.phpt?view=markup&rev=1.1
Index: php-src/ext/pcre/tests/invalid_utf8.ph
tony2001Mon May 29 20:31:48 2006 UTC
Added files: (Branch: PHP_5_2)
/php-src/ext/pcre/tests invalid_utf8.phpt
Log:
add test
http://cvs.php.net/viewcvs.cgi/php-src/ext/pcre/tests/invalid_utf8.phpt?view=markup&rev=1.1
Index: php-src/ext/pcre/test
tony2001Mon May 29 21:08:02 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/mbstring/tests common.inc mb_strlen.phpt
Log:
fix tests
http://cvs.php.net/viewcvs.cgi/php-src/ext/mbstring/tests/common.inc?r1=1.2&r2=1.2.6.1&diff_format=u
Index: php-
@@ -1115,6 +1115,10 @@
}
} else {
pcre_handle_exec_error(count TSRMLS_CC);
+ if (result) {
+ efree(result);
+ result = NULL;
+ }
break;
}
Nice catch, Tony! But you don't need the if, because result is never NULL
(it is assigned only from safe_malloc()).
Nuno
--
PHP CVS Mailing List (h
tony2001Mon May 29 21:31:30 2006 UTC
Modified files:
/php-src/ext/pcre php_pcre.c
Log:
no need to check it for not-NULL (as was pointed by Nuno)
http://cvs.php.net/viewcvs.cgi/php-src/ext/pcre/php_pcre.c?r1=1.182&r2=1.183&diff_format=u
Index: php-sr
tony2001Mon May 29 21:31:49 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/pcre php_pcre.c
Log:
MFH: no need to check it for not-NULL
http://cvs.php.net/viewcvs.cgi/php-src/ext/pcre/php_pcre.c?r1=1.168.2.9.2.3&r2=1.168.2.9.2.4&diff_format=u
On 30.05.2006 01:11, Nuno Lopes wrote:
@@ -1115,6 +1115,10 @@
}
} else {
pcre_handle_exec_error(count TSRMLS_CC);
+ if (result) {
+ efree(result);
+ result = NULL;
+ }
break;
}
Nice catch, Tony! But you don't need the if, because result is never NULL
(it is assigned only from safe_malloc(
iliaa Tue May 30 02:40:33 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/wddx wddx.c
Log:
Convert E_ERROR to E_RECOVERABLE_ERROR
http://cvs.php.net/viewcvs.cgi/php-src/ext/wddx/wddx.c?r1=1.119.2.10.2.2&r2=1.119.2.10.2.3&diff_format=u
Index: php-sr
iliaa Tue May 30 02:40:49 2006 UTC
Modified files:
/php-src/ext/wddx wddx.c
Log:
MFB: Convert E_ERROR to E_RECOVERABLE_ERROR
http://cvs.php.net/viewcvs.cgi/php-src/ext/wddx/wddx.c?r1=1.135&r2=1.136&diff_format=u
Index: php-src/ext/wddx/wddx.c
diff -u php-
27 matches
Mail list logo
