MFB: Refactor code to make fixes for coverity issues #385 and #386 a bit

more obvious

diff --git a/Zend/zend_ini.c b/Zend/zend_ini.c
index 7028febf23c47aa509993e6fa80a0b99d435e175..f33ad84d55480acafad1a2513e1199d9690b01f4 100644 (file)
--- a/Zend/zend_ini.c
+++ b/Zend/zend_ini.c
@@ -438,14 +438,16 @@ ZEND_INI_DISP(zend_ini_boolean_displayer_cb) /* {{{ */
                tmp_value_len = 0;
        }
 
-       if (tmp_value_len == 4 && strcasecmp(tmp_value, "true") == 0) {
-               value = 1;
-       } else if (tmp_value_len == 3 && strcasecmp(tmp_value, "yes") == 0) {
-               value = 1;
-       } else if (tmp_value_len == 2 && strcasecmp(tmp_value, "on") == 0) {
-               value = 1;
-       } else if (tmp_value) {
-               value = atoi(tmp_value);
+       if (tmp_value) {
+               if (tmp_value_len == 4 && strcasecmp(tmp_value, "true") == 0) {
+                       value = 1;
+               } else if (tmp_value_len == 3 && strcasecmp(tmp_value, "yes") == 0) {
+                       value = 1;
+               } else if (tmp_value_len == 2 && strcasecmp(tmp_value, "on") == 0) {
+                       value = 1;
+               } else (tmp_value) {
+                       value = atoi(tmp_value);
+               }
        } else {
                value = 0;
        }

diff --git a/main/main.c b/main/main.c
index 85b5113a0d62aaad0e0a2acbbafa58c88cb7281d..9ef702f123beefa1b02571e2522c4d26acc9bd45 100644 (file)
--- a/main/main.c
+++ b/main/main.c
@@ -338,6 +338,10 @@ static int php_get_display_errors_mode(char *value, int value_length)
 {
        int mode;
 
+       if (!value) {
+               return PHP_DISPLAY_ERRORS_STDOUT;
+       }
+
        if (value_length == 2 && !strcasecmp("on", value)) {
                mode = PHP_DISPLAY_ERRORS_STDOUT;
        } else if (value_length == 3 && !strcasecmp("yes", value)) {
@@ -348,13 +352,11 @@ static int php_get_display_errors_mode(char *value, int value_length)
                mode = PHP_DISPLAY_ERRORS_STDERR;
        } else if (value_length == 6 && !strcasecmp(value, "stdout")) {
                mode = PHP_DISPLAY_ERRORS_STDOUT;
-       } else if (value ){
+       } else {
                mode = atoi(value);
                if (mode && mode != PHP_DISPLAY_ERRORS_STDOUT && mode != PHP_DISPLAY_ERRORS_STDERR) {
                        mode = PHP_DISPLAY_ERRORS_STDOUT;
                }
-       } else {
-               mode = PHP_DISPLAY_ERRORS_STDOUT;
        }
        return mode;
 }