iliaa           Sat Jan 18 15:01:48 2003 EDT

  Modified files:              
    /php4/ext/standard  exec.c file.c image.c info.c metaphone.c 
                        string.c 
  Log:
  Removed pointless memory allocation checks.
  
  
Index: php4/ext/standard/exec.c
diff -u php4/ext/standard/exec.c:1.91 php4/ext/standard/exec.c:1.92
--- php4/ext/standard/exec.c:1.91       Wed Jan 15 11:29:00 2003
+++ php4/ext/standard/exec.c    Sat Jan 18 15:01:40 2003
@@ -15,7 +15,7 @@
    | Author: Rasmus Lerdorf                                               |
    +----------------------------------------------------------------------+
  */
-/* $Id: exec.c,v 1.91 2003/01/15 16:29:00 wez Exp $ */
+/* $Id: exec.c,v 1.92 2003/01/18 20:01:40 iliaa Exp $ */
 
 #include <stdio.h>
 #include "php.h"
@@ -67,10 +67,6 @@
 #endif
 
        buf = (char *) emalloc(EXEC_INPUT_BUF);
-       if (!buf) {
-               php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to emalloc %d 
bytes for exec buffer", EXEC_INPUT_BUF);
-               return -1;
-       }
        buflen = EXEC_INPUT_BUF;
 
        if (PG(safe_mode)) {
@@ -162,14 +158,6 @@
                        do {
                                if ( buflen <= (l+1) ) {
                                        buf = erealloc(buf, buflen + EXEC_INPUT_BUF);
-                                       if ( buf == NULL ) {
-                                               php_error_docref(NULL TSRMLS_CC, 
E_WARNING, "Unable to erealloc %d bytes for exec buffer", 
-                                                               buflen + 
EXEC_INPUT_BUF);
-#if PHP_SIGCHILD
-                                               signal (SIGCHLD, sig_handler);
-#endif
-                                               return -1;
-                                       }
                                        buflen += EXEC_INPUT_BUF;
                                }
 
Index: php4/ext/standard/file.c
diff -u php4/ext/standard/file.c:1.294 php4/ext/standard/file.c:1.295
--- php4/ext/standard/file.c:1.294      Sat Jan 18 09:10:22 2003
+++ php4/ext/standard/file.c    Sat Jan 18 15:01:41 2003
@@ -21,7 +21,7 @@
    +----------------------------------------------------------------------+
  */
 
-/* $Id: file.c,v 1.294 2003/01/18 14:10:22 wez Exp $ */
+/* $Id: file.c,v 1.295 2003/01/18 20:01:41 iliaa Exp $ */
 
 /* Synced with php 3.0 revision 1.218 1999-06-16 [ssb] */
 
@@ -1495,7 +1495,7 @@
                WRONG_PARAM_COUNT;
        }
        args = (zval ***)emalloc(argCount * sizeof(zval **));
-       if (!args || (zend_get_parameters_array_ex(argCount, args) == FAILURE)) {
+       if (zend_get_parameters_array_ex(argCount, args) == FAILURE) {
                efree( args );
                WRONG_PARAM_COUNT;
        }
Index: php4/ext/standard/image.c
diff -u php4/ext/standard/image.c:1.84 php4/ext/standard/image.c:1.85
--- php4/ext/standard/image.c:1.84      Fri Jan 17 13:51:30 2003
+++ php4/ext/standard/image.c   Sat Jan 18 15:01:43 2003
@@ -17,7 +17,7 @@
    +----------------------------------------------------------------------+
  */
 
-/* $Id: image.c,v 1.84 2003/01/17 18:51:30 helly Exp $ */
+/* $Id: image.c,v 1.85 2003/01/18 20:01:43 iliaa Exp $ */
 
 #include "php.h"
 #include <stdio.h>
@@ -215,10 +215,6 @@
                do {
                        szlength=slength*(1<<factor++);
                        buf = (char *) erealloc(buf,szlength);
-                       if (!buf) {
-                               status = 1;
-                               break;
-                       } 
                        status = uncompress(buf, &szlength, bufz, slength);
                } while ((status==Z_BUF_ERROR)&&(factor<maxfactor));
                
@@ -439,7 +435,6 @@
        length -= 2;                            /* length includes itself */
 
        buffer = emalloc(length);
-       if ( !buffer) return;
 
        if (php_stream_read(stream, buffer, (long) length) <= 0) {
                efree(buffer);
@@ -485,8 +480,6 @@
                                if (result == NULL) {
                                        /* handle SOFn block */
                                        result = (struct gfxinfo *) ecalloc(1, 
sizeof(struct gfxinfo));
-                                       if ( !result)
-                                               return NULL;
                                        length = php_read2(stream TSRMLS_CC);
                                        result->bits     = php_stream_getc(stream);
                                        result->height   = php_read2(stream TSRMLS_CC);
@@ -605,9 +598,6 @@
        }
 
        result = (struct gfxinfo *)ecalloc(1, sizeof(struct gfxinfo));
-       if (!result) {
-               return NULL;
-       }
 
        dummy_short = php_read2(stream TSRMLS_CC); /* Lsiz */
        dummy_short = php_read2(stream TSRMLS_CC); /* Rsiz */
Index: php4/ext/standard/info.c
diff -u php4/ext/standard/info.c:1.223 php4/ext/standard/info.c:1.224
--- php4/ext/standard/info.c:1.223      Fri Jan 17 13:07:10 2003
+++ php4/ext/standard/info.c    Sat Jan 18 15:01:43 2003
@@ -18,7 +18,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: info.c,v 1.223 2003/01/17 18:07:10 derick Exp $ */
+/* $Id: info.c,v 1.224 2003/01/18 20:01:43 iliaa Exp $ */
 
 #include "php.h"
 #include "php_ini.h"
@@ -439,10 +439,7 @@
                                for 
(zend_hash_internal_pointer_reset(url_stream_wrappers_hash);
                                                
zend_hash_get_current_key_ex(url_stream_wrappers_hash, &stream_protocol, 
&stream_protocol_len, NULL, 0, NULL) == HASH_KEY_IS_STRING;
                                                
zend_hash_move_forward(url_stream_wrappers_hash)) {
-                                       if (NULL == (stream_protocols_buf = 
erealloc(stream_protocols_buf,
-                                                                       
stream_protocols_buf_len + stream_protocol_len + 2 /* ", " */ + 1 /* 0 byte at end 
*/))) {
-                                               break;
-                                       }
+                                       stream_protocols_buf = 
+erealloc(stream_protocols_buf, stream_protocols_buf_len + stream_protocol_len + 2 + 
+1);
                                        memcpy(stream_protocols_buf + 
stream_protocols_buf_len, stream_protocol, stream_protocol_len);
                                        stream_protocols_buf[stream_protocols_buf_len 
+ stream_protocol_len] = ',';
                                        stream_protocols_buf[stream_protocols_buf_len 
+ stream_protocol_len + 1] = ' ';
Index: php4/ext/standard/metaphone.c
diff -u php4/ext/standard/metaphone.c:1.22 php4/ext/standard/metaphone.c:1.23
--- php4/ext/standard/metaphone.c:1.22  Tue Dec 31 11:07:48 2002
+++ php4/ext/standard/metaphone.c       Sat Jan 18 15:01:46 2003
@@ -16,7 +16,7 @@
    +----------------------------------------------------------------------+
 */
 
-/* $Id: metaphone.c,v 1.22 2002/12/31 16:07:48 sebastian Exp $ */
+/* $Id: metaphone.c,v 1.23 2003/01/18 20:01:46 iliaa Exp $ */
 
 /*
        Based on CPANs "Text-Metaphone-1.96" by Michael G Schwern <[EMAIL PROTECTED]> 
@@ -145,9 +145,7 @@
  * could be one though; or more too). */
 #define Phonize(c)     { \
                                                if (p_idx >= max_buffer_len) { \
-                                                       if (NULL == (*phoned_word = 
erealloc(*phoned_word, max_buffer_len + 2))) { \
-                                                               return -1; \
-                                                       } \
+                                                       *phoned_word = 
+erealloc(*phoned_word, max_buffer_len + 2); \
                                                        max_buffer_len += 2; \
                                                } \
                                                (*phoned_word)[p_idx++] = c; \
@@ -185,13 +183,9 @@
        if (max_phonemes == 0) {        /* Assume largest possible */
                max_buffer_len = word_len;
                *phoned_word = emalloc(sizeof(char) * word_len + 1);
-               if (!*phoned_word)
-                       return -1;
        } else {
                max_buffer_len = max_phonemes;
                *phoned_word = emalloc(sizeof(char) * max_phonemes + 1);
-               if (!*phoned_word)
-                       return -1;
        }
 
 
Index: php4/ext/standard/string.c
diff -u php4/ext/standard/string.c:1.346 php4/ext/standard/string.c:1.347
--- php4/ext/standard/string.c:1.346    Sat Jan 11 18:05:19 2003
+++ php4/ext/standard/string.c  Sat Jan 18 15:01:46 2003
@@ -18,7 +18,7 @@
    +----------------------------------------------------------------------+
  */
 
-/* $Id: string.c,v 1.346 2003/01/11 23:05:19 moriyoshi Exp $ */
+/* $Id: string.c,v 1.347 2003/01/18 20:01:46 iliaa Exp $ */
 
 /* Synced with php 3.0 revision 1.193 1999-06-16 [ssb] */
 
@@ -113,9 +113,6 @@
        size_t i, j;
 
        result = (char *) emalloc(oldlen * 2 * sizeof(char) + 1);
-       if (!result) {
-               return result;
-       }
        
        for (i = j = 0; i < oldlen; i++) {
                result[j++] = hexconvtab[old[i] >> 4];



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

Reply via email to