Commit:    d0c40220d1ba1d3d1e50ca122d1ec37e2d19c58a
Author:    Remi Collet <r...@php.net>         Thu, 2 May 2013 10:32:45 +0200
Parents:   8492bbefbb74ee0cd009884f6ee82c409ffbe018
Branches:  PHP-5.3 PHP-5.4 PHP-5.5 master

Link:       
http://git.php.net/?p=php-src.git;a=commitdiff;h=d0c40220d1ba1d3d1e50ca122d1ec37e2d19c58a

Log:
fixed size array cannot be null

Changed paths:
  M  sapi/fpm/fpm/fpm_log.c


Diff:
diff --git a/sapi/fpm/fpm/fpm_log.c b/sapi/fpm/fpm/fpm_log.c
index 6b014b5..303acb2 100644
--- a/sapi/fpm/fpm/fpm_log.c
+++ b/sapi/fpm/fpm/fpm_log.c
@@ -237,7 +237,7 @@ int fpm_log_write(char *log_format TSRMLS_DC) /* {{{ */
 
                                case 'f': /* script */
                                        if (!test) {
-                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s",  proc.script_filename && *proc.script_filename ? 
proc.script_filename : "-");
+                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s",  *proc.script_filename ? proc.script_filename : 
"-");
                                        }
                                        break;
 
@@ -249,7 +249,7 @@ int fpm_log_write(char *log_format TSRMLS_DC) /* {{{ */
 
                                case 'm': /* method */
                                        if (!test) {
-                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.request_method && *proc.request_method ? 
proc.request_method : "-");
+                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", *proc.request_method ? proc.request_method : "-");
                                        }
                                        break;
 
@@ -347,19 +347,19 @@ int fpm_log_write(char *log_format TSRMLS_DC) /* {{{ */
 
                                case 'q': /* query_string */
                                        if (!test) {
-                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.query_string ? proc.query_string : "");
+                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.query_string);
                                        }
                                        break;
 
                                case 'Q': /* '?' */
                                        if (!test) {
-                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.query_string && *proc.query_string  ? "?" : 
"");
+                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", *proc.query_string  ? "?" : "");
                                        }
                                        break;
 
                                case 'r': /* request URI */
                                        if (!test) {
-                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.request_uri ? proc.request_uri : "-");
+                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.request_uri);
                                        }
                                        break;
 
@@ -397,7 +397,7 @@ int fpm_log_write(char *log_format TSRMLS_DC) /* {{{ */
 
                                case 'u': /* remote user */
                                        if (!test) {
-                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.auth_user ? proc.auth_user : "-");
+                                               len2 = snprintf(b, 
FPM_LOG_BUFFER - len, "%s", proc.auth_user);
                                        }
                                        break;


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

Reply via email to