BUG/MINOR: contrib/modsecurity: close the va_list ap before return
Make sure the va_list ap is closed before return in case the function
vsnprintf() returned an error.
diff --git a/contrib/modsecurity/modsec_wrapper.c b/contrib/modsecurity/modsec_wrapper.c
index 8927b5b..271ec15 100644
--- a/contrib/modsecurity/modsec_wrapper.c
+++ b/contrib/modsecurity/modsec_wrapper.c
@@ -105,9 +105,10 @@
va_start(ap, fmt);
len = vsnprintf(NULL, 0, fmt, ap);
+ va_end(ap);
+
if (len == -1)
return NULL;
- va_end(ap);
out = apr_pcalloc(req->pool, len + 1);
if (!out)
@@ -115,9 +116,10 @@
va_start(ap, fmt);
len = vsnprintf(out, len + 1, fmt, ap);
+ va_end(ap);
+
if (len == -1)
return NULL;
- va_end(ap);
return out;
}