From: Xinchen Hui <laruence@php.net>
Date: Sun, 7 Aug 2011 13:19:04 +0000 (+0000)
Subject: Fixed bug that may dereferenced NULL pointer before checking
X-Git-Tag: php-5.3.7RC5~39
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b2110c3eea85072e5c2a8a1aab363eb727c4a5d3;p=php

Fixed bug that may dereferenced NULL pointer before checking
---

diff --git a/ext/standard/basic_functions.c b/ext/standard/basic_functions.c
index d1cd20e86e..aee53af2d9 100644
--- a/ext/standard/basic_functions.c
+++ b/ext/standard/basic_functions.c
@@ -4313,15 +4313,15 @@ PHP_FUNCTION(getopt)
 		/* the first <len> slots are filled by the one short ops
 		 * we now extend our array and jump to the new added structs */
 		opts = (opt_struct *) erealloc(opts, sizeof(opt_struct) * (len + count + 1));
+		if (!opts) {
+			RETURN_FALSE;
+		}
+
 		orig_opts = opts;
 		opts += len;
 
 		memset(opts, 0, count * sizeof(opt_struct));
 
-		if (!opts) {
-			RETURN_FALSE;
-		}
-
 		/* Reset the array indexes. */
 		zend_hash_internal_pointer_reset(Z_ARRVAL_P(p_longopts));
 
@@ -4358,6 +4358,10 @@ PHP_FUNCTION(getopt)
 		}
 	} else {
 		opts = (opt_struct*) erealloc(opts, sizeof(opt_struct) * (len + 1));
+		if (!opts) {
+			RETURN_FALSE;
+		}
+
 		orig_opts = opts;
 		opts += len;
 	}