diff options
author | Xinchen Hui <laruence@php.net> | 2011-08-07 14:25:30 +0000 |
---|---|---|
committer | Xinchen Hui <laruence@php.net> | 2011-08-07 14:25:30 +0000 |
commit | 2ce3f11f4bd20548dfdb27366fb2dd8585d6ec2a (patch) | |
tree | ce7ba6edca42f7293765877d8ee0345d6d4d61bb | |
parent | a35462c71cf084d169100bc1bc0de4e3e69f00cf (diff) | |
download | php-git-2ce3f11f4bd20548dfdb27366fb2dd8585d6ec2a.tar.gz |
revert -r314422
No need to check erealloc's return
-rw-r--r-- | ext/standard/basic_functions.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/ext/standard/basic_functions.c b/ext/standard/basic_functions.c index cca7d8385a..3b604efe2a 100644 --- a/ext/standard/basic_functions.c +++ b/ext/standard/basic_functions.c @@ -4282,15 +4282,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)); @@ -4327,10 +4327,6 @@ PHP_FUNCTION(getopt) } } else { opts = (opt_struct*) erealloc(opts, sizeof(opt_struct) * (len + 1)); - if (!opts) { - RETURN_FALSE; - } - orig_opts = opts; opts += len; } |