Avoid some useless code and memory allocation in case no macro is present

Marc Stern · Marc Stern · commit 40c6f80ccffb · 2023-10-03T16:50:12.000+02:00
diff --git a/apache2/re_actions.c b/apache2/re_actions.c
@@ -273,6 +273,7 @@ int expand_macros(modsec_rec *msr, msc_string *var, msre_rule *rule, apr_pool_t
                 next_text_start = p + 1;
             }
         } else {
+            if (arr->nelts == 0) return 0; /* no macro */
             /* Text part. */
             part = (msc_string *)apr_pcalloc(mptmp, sizeof(msc_string));
             part->value = apr_pstrdup(mptmp, text_start);

Original file line number	Diff line number	Diff line change
`@@ -273,6 +273,7 @@ int expand_macros(modsec_rec msr, msc_string var, msre_rule *rule, apr_pool_t`
`273`	`273`	`next_text_start = p + 1;`
`274`	`274`	`}`
`275`	`275`	`} else {`
	`276`	`+ if (arr->nelts == 0) return 0; /* no macro */`
`276`	`277`	`/* Text part. */`
`277`	`278`	`part = (msc_string *)apr_pcalloc(mptmp, sizeof(msc_string));`
`278`	`279`	`part->value = apr_pstrdup(mptmp, text_start);`