--- parser3/src/targets/cgi/parser3.C 2019/12/27 20:46:26 1.285 +++ parser3/src/targets/cgi/parser3.C 2020/08/12 17:30:40 1.287 @@ -5,7 +5,7 @@ Author: Alexandr Petrosian (http://paf.design.ru) */ -volatile const char * IDENT_PARSER3_C="$Id: parser3.C,v 1.285 2019/12/27 20:46:26 moko Exp $"; +volatile const char * IDENT_PARSER3_C="$Id: parser3.C,v 1.287 2020/08/12 17:30:40 moko Exp $"; #include "pa_config_includes.h" @@ -113,8 +113,8 @@ static void log(const char* fmt, va_list if(request_info) fprintf(f, " [uri=%s, method=%s, cl=%lu]", - request_info->uri? request_info->uri: "", - request_info->method? request_info->method: "", + request_info->uri ? request_info->uri : "", + request_info->method ? request_info->method : "", request_info->content_length); else fputs(" [no request info]", f); @@ -272,15 +272,9 @@ static void full_file_spec(const char* f static void log_signal(const char* signal_name) { if(request_info) - SAPI::log(SAPI_info, "%s received while %s. uri=%s, method=%s, cl=%u", - signal_name, - request ? "executing code" : "reading data", - request_info->uri, - request_info->method, - request_info->content_length); + SAPI::log(SAPI_info, "%s received while %s.", signal_name, request ? "executing code" : "reading data"); else - SAPI::log(SAPI_info, "%s received before or after processing request", - signal_name); + SAPI::log(SAPI_info, "%s received before or after processing request", signal_name); } #ifdef SIGUSR1 @@ -423,7 +417,7 @@ static void real_parser_handler(const ch SAPI::die("CGI: illegal call (1)"); } else { // fcgiwrap minimalistic setup - if(request_info.query_string) { + if(request_info.query_string && *request_info.query_string) { char* reconstructed_uri = new(PointerFreeGC) char[strlen(path_info) + 1/*'?'*/+ strlen(request_info.query_string) + 1/*0*/]; strcpy(reconstructed_uri, path_info); strcat(reconstructed_uri, "?");