You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rj...@apache.org on 2017/10/16 21:41:15 UTC

svn commit: r1812332 - /httpd/httpd/trunk/modules/metadata/mod_remoteip.c

Author: rjung
Date: Mon Oct 16 21:41:15 2017
New Revision: 1812332

URL: http://svn.apache.org/viewvc?rev=1812332&view=rev
Log:
Fix format pattern (%lu => %APR_SIZE_T_FMT).

Detected by maintainer mode compilation and GCC error:

.../modules/metadata/mod_remoteip.c:
In function 'remoteip_input_filter':
.../include/http_log.h:117:33:
error: format '%lu' expects argument of type
'long unsigned int', but argument 8 has type
'apr_size_t {aka unsigned int}' [-Werror=format=]

Modified:
    httpd/httpd/trunk/modules/metadata/mod_remoteip.c

Modified: httpd/httpd/trunk/modules/metadata/mod_remoteip.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/metadata/mod_remoteip.c?rev=1812332&r1=1812331&r2=1812332&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/metadata/mod_remoteip.c (original)
+++ httpd/httpd/trunk/modules/metadata/mod_remoteip.c Mon Oct 16 21:41:15 2017
@@ -1149,8 +1149,9 @@ static apr_status_t remoteip_input_filte
     if (ctx->rcvd > ctx->need || !APR_BRIGADE_EMPTY(ctx->bb)) {
         ap_log_cerror(APLOG_MARK, APLOG_ERR, 0, f->c, APLOGNO(03513)
                       "RemoteIPProxyProtocol: internal error: have data left over; "
-                      " need=%lu, rcvd=%lu, brigade-empty=%d", ctx->need,
-                      ctx->rcvd, APR_BRIGADE_EMPTY(ctx->bb));
+                      " need=%" APR_SIZE_T_FMT ", rcvd=%" APR_SIZE_T_FMT
+                      ", brigade-empty=%d", ctx->need, ctx->rcvd,
+                      APR_BRIGADE_EMPTY(ctx->bb));
         f->c->aborted = 1;
         apr_brigade_destroy(ctx->bb);
         return APR_ECONNABORTED;