Return early

Author: Girgias

ext/bz2/bz2_filter.c

@@ -361,6 +361,7 @@ static php_stream_filter *php_bz2_filter_create(const char *filtername, zval *fi
 					zend_long blocks = zval_get_long(tmpzval);
 					if (blocks < 1 || blocks > 9) {
 						zend_value_error("Invalid parameter given for number of blocks to allocate. (" ZEND_LONG_FMT ")", blocks);
+						return NULL;
 					} else {
 						blockSize100k = (int) blocks;
 					}
@@ -371,6 +372,7 @@ static php_stream_filter *php_bz2_filter_create(const char *filtername, zval *fi
 					zend_long work = zval_get_long(tmpzval);
 					if (work < 0 || work > 250) {
 						zend_value_error("Invalid parameter given for work factor. (" ZEND_LONG_FMT ")", work);
+						return NULL;
 					} else {
 						workFactor = (int) work;
 					}

ext/bz2/tests/bug72447.phpt

@@ -22,5 +22,6 @@ fclose($fp);
 <?php
 unlink('testfile');
 ?>
---EXPECT--
+--EXPECTF--
+Warning: stream_filter_append(): unable to create or locate filter "bzip2.compress" %s on line %d
 Invalid parameter given for number of blocks to allocate. (0)