ASPN ActiveState Programmer Network
ActiveState
/ Home / Perl / PHP / Python / Tcl / XSLT /
/ Safari / My ASPN /
Cookbooks | Documentation | Mailing Lists | Modules | News Feeds | Products | User Groups


Recent Messages
List Archives
About the List
List Leaders
Subscription Options

View Subscriptions
Help

View by Topic
ActiveState
.NET Framework
Open Source
Perl
PHP
Python
Tcl
Web Services
XML & XSLT

View by Category
Database
General
SOAP
System Administration
Tools
User Interfaces
Web Programming
XML Programming


MyASPN >> Mail Archive >> php-cvs
php-cvs
[PHP-CVS] cvs: php4 /ext/bz2 bz2.c
by Ilia Alshanetsky other posts by this author
Aug 29 2002 11:45AM messages near this date
[PHP-CVS] cvs: php4 /ext/mcrypt/tests 001.phpt | [PHP-CVS] cvs: php4 /main php_ini.c
iliaa		Thu Aug 29 07:45:29 2002 EDT

  Modified files:              
    /php4/ext/bz2	bz2.c 
  Log:
  Added a check for a negative return value of php_stream_read, which if 
  left uncheck can result in segv.
  
  
Index: php4/ext/bz2/bz2.c
diff -u php4/ext/bz2/bz2.c:1.56 php4/ext/bz2/bz2.c:1.57
--- php4/ext/bz2/bz2.c:1.56	Sat Aug 17 09:15:05 2002
+++ php4/ext/bz2/bz2.c	Thu Aug 29 07:45:27 2002
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
  */
  
-/* $Id: bz2.c,v 1.56 2002/08/17 13:15:05 kalowsky Exp $ */
+/* $Id: bz2.c,v 1.57 2002/08/29 11:45:27 iliaa Exp $ */
 
 #ifdef HAVE_CONFIG_H
 #include "config.h"
@@ -266,6 +266,13 @@
 
 	Z_STRVAL_P(return_value) = emalloc(len + 1);
 	Z_STRLEN_P(return_value) = php_stream_read(stream, Z_STRVAL_P(return_value), len);
+	
+	if (Z_STRLEN_P(return_value) < 0) {
+		efree(Z_STRVAL_P(return_value));
+		zend_error(E_WARNING, "%s(): could not read valid bz2 data from stream", get_active_funct
ion_name(TSRMLS_C));
+		RETURN_FALSE;		
+	}
+	
 	Z_STRVAL_P(return_value)[Z_STRLEN_P(return_value)] = 0;
 
 	if (PG(magic_quotes_runtime)) {



-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Privacy Policy | Email Opt-out | Feedback | Syndication
© ActiveState Software Inc. All rights reserved