iliaa Sun Dec 31 22:26:06 2006 UTC
Modified files:
/php-src/ext/session session.c
Log:
MFB: Added boundary checks to php_binary deserializer
http://cvs.php.net/viewvc.cgi/php-src/ext/session/session.c?r1=1.459&r2=1.460&diff_format=u
Index: php-src/ext/session/session.c
diff -u php-src/ext/session/session.c:1.459 php-src/ext/session/session.c:1.460
--- php-src/ext/session/session.c:1.459 Tue Dec 26 17:18:28 2006
+++ php-src/ext/session/session.c Sun Dec 31 22:26:06 2006
@@ -17,7 +17,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: session.c,v 1.459 2006/12/26 17:18:28 iliaa Exp $ */
+/* $Id: session.c,v 1.460 2006/12/31 22:26:06 iliaa Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -384,6 +384,11 @@
for (p = val; p < endptr; ) {
zval **tmp;
namelen = *p & (~PS_BIN_UNDEF);
+
+ if (namelen > PS_BIN_MAX || (p + namelen) >= endptr) {
+ return FAILURE;
+ }
+
has_value = *p & PS_BIN_UNDEF ? 0 : 1;
name = estrndup(p + 1, namelen);
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
