Commit 21efe76d authored by Dridi Boukelmoune's avatar Dridi Boukelmoune

Appropriate VSL group checking

We have a dedicated enum entry for that, and nothing prevents the caller
from passing a negative value.
parent 2962375b
...@@ -1088,7 +1088,7 @@ VSLQ_New(struct VSL_data *vsl, struct VSL_cursor **cp, ...@@ -1088,7 +1088,7 @@ VSLQ_New(struct VSL_data *vsl, struct VSL_cursor **cp,
struct VSLQ *vslq; struct VSLQ *vslq;
CHECK_OBJ_NOTNULL(vsl, VSL_MAGIC); CHECK_OBJ_NOTNULL(vsl, VSL_MAGIC);
if (grouping > VSL_g_session) { if (grouping < 0 || grouping >= VSL_g__MAX) {
(void)vsl_diag(vsl, "Illegal query grouping"); (void)vsl_diag(vsl, "Illegal query grouping");
return (NULL); return (NULL);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment