Skip to content

Commit 1e63e49

Browse files
author
Felipe Zimmerle
committed
Uses %zu to print size_t instead of %d.
This will help to reduce the amount of warnings during the compilation See #631 for further information.
1 parent 503e8f6 commit 1e63e49

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

apache2/msc_crypt.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1103,7 +1103,7 @@ int inject_hashed_response_body(modsec_rec *msr, int elts) {
11031103
memcpy(msr->stream_output_data, xmlOutputBufferGetContent(output_buf), msr->stream_output_length);
11041104

11051105
if (msr->txcfg->debuglog_level >= 4)
1106-
msr_log(msr, 4, "inject_hashed_response_body: Copying XML tree from CONTENT to stream buffer [%d] bytes.", xmlOutputBufferGetSize(output_buf));
1106+
msr_log(msr, 4, "inject_hashed_response_body: Copying XML tree from CONTENT to stream buffer [%zu] bytes.", xmlOutputBufferGetSize(output_buf));
11071107

11081108
} else {
11091109

@@ -1132,7 +1132,7 @@ int inject_hashed_response_body(modsec_rec *msr, int elts) {
11321132
memcpy(msr->stream_output_data, xmlOutputBufferGetContent(output_buf), msr->stream_output_length);
11331133

11341134
if (msr->txcfg->debuglog_level >= 4)
1135-
msr_log(msr, 4, "inject_hashed_response_body: Copying XML tree from CONV to stream buffer [%d] bytes.", xmlOutputBufferGetSize(output_buf));
1135+
msr_log(msr, 4, "inject_hashed_response_body: Copying XML tree from CONV to stream buffer [%zu] bytes.", xmlOutputBufferGetSize(output_buf));
11361136

11371137
}
11381138

@@ -1379,7 +1379,7 @@ char *do_hash_link(modsec_rec *msr, char *link, int type) {
13791379
relative_link = relative_uri+1;
13801380

13811381
if (msr->txcfg->debuglog_level >= 4) {
1382-
msr_log(msr, 4, "Signing data [%s] size %d", relative_link, strlen(relative_link));
1382+
msr_log(msr, 4, "Signing data [%s] size %zu", relative_link, strlen(relative_link));
13831383
}
13841384

13851385
if(msr->txcfg->crypto_key_add == HASH_KEYONLY)

apache2/msc_multipart.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ static int multipart_parse_content_disposition(modsec_rec *msr, char *c_d_value)
227227
p--;
228228
if(*p == '\'' || *p == '\"') {
229229
if (msr->txcfg->debuglog_level >= 9) {
230-
msr_log(msr, 9, "Multipart: Invalid quoting detected: %s length %d bytes",
230+
msr_log(msr, 9, "Multipart: Invalid quoting detected: %s length %zu bytes",
231231
log_escape_nq(msr->mp, p), strlen(p));
232232
}
233233
msr->mpd->flag_invalid_quoting = 1;

apache2/re_operators.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -843,7 +843,7 @@ static int msre_op_validateHash_execute(modsec_rec *msr, msre_rule *rule, msre_v
843843

844844
nlink = apr_pstrmemdup(msr->mp, target, strlen(target) - strlen(valid) - 1);
845845

846-
msr_log(msr, 9, "Validating URI %s size %d",nlink,strlen(nlink));
846+
msr_log(msr, 9, "Validating URI %s size %zu",nlink,strlen(nlink));
847847

848848
hash_link = do_hash_link(msr, (char *)nlink, HASH_ONLY);
849849

0 commit comments

Comments
 (0)