Skip to content
Permalink
Browse files

Appropriate VSL group checking

We have a dedicated enum entry for that, and nothing prevents the caller
from passing a negative value.
  • Loading branch information...
Dridi committed May 16, 2019
1 parent 2962375 commit 21efe76d23ed81e5423c59055e55c2c5a7f29dbd
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/libvarnishapi/vsl_dispatch.c
@@ -1088,7 +1088,7 @@ VSLQ_New(struct VSL_data *vsl, struct VSL_cursor **cp,
struct VSLQ *vslq;

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");
return (NULL);
}

0 comments on commit 21efe76

Please sign in to comment.
You can’t perform that action at this time.