Skip to content

Commit a235b53

Browse files
author
Felipe Zimmerle
committed
Fix apr_crypto.h include in modsecurity.h and msc_remote_rules.c
apr_crypto is not always available, configure scripts are looking for it and setting WITH_APU_CRYPTO in case it is found. There were two places where apr_crypto.h was included without validating if WITH_APU_CRYPTO was set. This patch removes the inclusion from the first place (it seems not to be mandatory) and add the check for WITH_APU_CRYPTO in the second one. This fix issue #834.
1 parent 2e09b8e commit a235b53

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

CHANGES

+7
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
1+
XX xxx XXXX - 2.9.0
2+
-------------------
3+
4+
* Fix apr_crypto.h include, now checking if apr_crypto.h is available by
5+
checking the definition WITH_APU_CRYPTO.
6+
[martinjina and ModSecurity team]
7+
18
15 Dez 2014 - 2.9.0-RC2
29
-----------------------
310

apache2/modsecurity.h

-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ typedef struct msc_parm msc_parm;
5252
#include "apr_md5.h"
5353
#include "apr_strings.h"
5454
#include "apr_hash.h"
55-
#include "apr_crypto.h"
5655
#include "httpd.h"
5756
#include "http_config.h"
5857
#include "http_log.h"

apache2/msc_remote_rules.c

+2
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,9 @@
2424
#include <apu.h>
2525

2626
#ifdef WITH_REMOTE_RULES
27+
#ifdef WITH_APU_CRYPTO
2728
#include <apr_crypto.h>
29+
#endif
2830
#include <apr_sha1.h>
2931
#endif
3032

0 commit comments

Comments
 (0)