summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGarrett Rooney <rooneg@apache.org>2006-04-28 19:50:56 +0200
committerGarrett Rooney <rooneg@apache.org>2006-04-28 19:50:56 +0200
commit30a1824be9012a7d567b312699a5f4124ec1e05a (patch)
tree2fe3f415d930e1fe5df43bff92c7b88585b18a0a
parent This resource /D syntax was pretty horridly imported into VS 2005, (diff)
downloadapache2-30a1824be9012a7d567b312699a5f4124ec1e05a.tar.xz
apache2-30a1824be9012a7d567b312699a5f4124ec1e05a.zip
Fix compile time warnings.
Pointed out by: jorton * modules/proxy/mod_proxy_fcgi.c Add include of util_script.h to pick up some function prototypes. (send_begin_request): Initialize the reserved bytes in the request body to zero. (dispatch): Use APR_SIZE_T_FMT instead of %d when logging an apr_size_t. git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@397968 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r--modules/proxy/mod_proxy_fcgi.c8
1 files changed, 7 insertions, 1 deletions
diff --git a/modules/proxy/mod_proxy_fcgi.c b/modules/proxy/mod_proxy_fcgi.c
index 51fb90533a..1d014632de 100644
--- a/modules/proxy/mod_proxy_fcgi.c
+++ b/modules/proxy/mod_proxy_fcgi.c
@@ -16,6 +16,7 @@
#include "mod_proxy.h"
#include "fcgi_protocol.h"
+#include "util_script.h"
module AP_MODULE_DECLARE_DATA proxy_fcgi_module;
@@ -240,6 +241,11 @@ static apr_status_t send_begin_request(proxy_conn_rec *conn, int request_id)
brb.roleB1 = ((FCGI_RESPONDER >> 8) & 0xff);
brb.roleB0 = ((FCGI_RESPONDER) & 0xff);
brb.flags = FCGI_KEEP_CONN;
+ brb.reserved[0] = 0;
+ brb.reserved[1] = 0;
+ brb.reserved[2] = 0;
+ brb.reserved[3] = 0;
+ brb.reserved[4] = 0;
fcgi_header_to_array(&header, farray);
fcgi_begin_request_body_to_array(&brb, abrb);
@@ -673,7 +679,7 @@ static apr_status_t dispatch(proxy_conn_rec *conn, request_rec *r,
if (readbuflen != FCGI_HEADER_LEN) {
ap_log_error(APLOG_MARK, APLOG_ERR, 0, r->server,
"proxy: FCGI: Failed to read entire header "
- "got %d wanted %d",
+ "got %" APR_SIZE_T_FMT " wanted %d",
readbuflen, FCGI_HEADER_LEN);
rv = APR_EINVAL;
break;