From: Pierre Joye <pierre.php@gmail.com>
Date: Fri, 16 May 2014 06:27:21 +0000 (+0200)
Subject: test the right length and remove unused var (remaining from 'old' version)
X-Git-Tag: POST_PHPNG_MERGE~339^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=797c1660cbd1e6ed0092ebbb153c9a2dec3054c8;p=php

test the right length and remove unused var (remaining from 'old' version)
---

diff --git a/ext/bz2/bz2.c b/ext/bz2/bz2.c
index bd4ac68d1d..db4e834a20 100644
--- a/ext/bz2/bz2.c
+++ b/ext/bz2/bz2.c
@@ -346,26 +346,24 @@ static PHP_FUNCTION(bzread)
 	long len = 1024;
 	php_stream *stream;
 	zend_string *data;
-	size_t dlen;
-	
+
 	if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r|l", &bz, &len)) {
 		RETURN_FALSE;
 	}
-	
+
 	php_stream_from_zval(stream, bz);
 
 	if ((len + 1) < 1) {
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "length may not be negative");
 		RETURN_FALSE;
 	}
-
 	data = STR_ALLOC(len, 0);
 	data->len = php_stream_read(stream, data->val, data->len);
-	
-	if (dlen < 0) {
+
+	if (data->len < 0) {
 		STR_FREE(data);
 		php_error_docref(NULL TSRMLS_CC, E_WARNING, "could not read valid bz2 data from stream");
-		RETURN_FALSE;		
+		RETURN_FALSE;
 	}
 	data->val[data->len] = '\0';