Skip to content

Commit 5f93853

Browse files
author
Marc Stern
committed
fixed a NULL check
1 parent c8e1904 commit 5f93853

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

apache2/re.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -873,6 +873,7 @@ static msre_action_metadata *msre_resolve_action(msre_engine *engine, const char
873873
msre_var *msre_create_var_ex(apr_pool_t *pool, msre_engine *engine, const char *name, const char *param,
874874
modsec_rec *msr, char **error_msg)
875875
{
876+
// msr can be NULL
876877
const char *varparam = param;
877878
msre_var *var = apr_pcalloc(pool, sizeof(msre_var));
878879
if (var == NULL) return NULL;
@@ -953,7 +954,7 @@ msre_var *msre_create_var_ex(apr_pool_t *pool, msre_engine *engine, const char *
953954
static msre_var *msre_create_var(msre_ruleset *ruleset, const char *name, const char *param,
954955
modsec_rec *msr, char **error_msg)
955956
{
956-
assert(msr != NULL);
957+
// msr can be NULL
957958
assert(ruleset != NULL);
958959
assert(error_msg != NULL);
959960
msre_var *var = msre_create_var_ex(ruleset->mp, ruleset->engine, name, param, msr, error_msg);

0 commit comments

Comments
 (0)