[master] 2d9fd0258 vre: Actually trigger the jit compilation

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Thu Aug 19 17:09:05 UTC 2021


commit 2d9fd0258dbb7d3426fb0e60b3e52d6f703a697a
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Thu Aug 19 17:15:52 2021 +0200

    vre: Actually trigger the jit compilation
    
    Spotted by Simon Vikström.
    
    Closes #3671

diff --git a/bin/varnishtest/tests/r01576.vtc b/bin/varnishtest/tests/r01576.vtc
index 7cc73e089..e866fb145 100644
--- a/bin/varnishtest/tests/r01576.vtc
+++ b/bin/varnishtest/tests/r01576.vtc
@@ -41,7 +41,7 @@ client c1 {
 } -run
 
 logexpect l1 -v v1 {
-	expect * * VCL_Error "(match|depth|recursion) limit exceeded"
+	expect * * VCL_Error "(match|depth|recursion|JIT stack) limit"
 } -start
 
 # This should fail with default params and JIT/no-JIT
diff --git a/lib/libvarnish/vre.c b/lib/libvarnish/vre.c
index 1e62184f9..e2efdcbc8 100644
--- a/lib/libvarnish/vre.c
+++ b/lib/libvarnish/vre.c
@@ -108,7 +108,7 @@ VRE_compile(const char *pattern, unsigned options,
 	}
 #if USE_PCRE2_JIT
 	if (jit)
-		(void)pcre2_jit_compile(v->re, 0);
+		(void)pcre2_jit_compile(v->re, PCRE2_JIT_COMPLETE);
 #else
 	(void)jit;
 #endif


More information about the varnish-commit mailing list