[master] 84b7dda I had forgotten that arg spec strings are NUL terminated centrally so I added one NUL too many to enum spec strings, preventing any subsequent args from being handled correctly.

Poul-Henning Kamp phk at varnish-cache.org
Thu Feb 17 20:16:35 CET 2011


commit 84b7dda89730901784afdaf48ac77e4a8d3d86e2
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Thu Feb 17 19:15:41 2011 +0000

    I had forgotten that arg spec strings are NUL terminated centrally
    so I added one NUL too many to enum spec strings, preventing any
    subsequent args from being handled correctly.
    
    Spotted by:	Nils Goroll

diff --git a/lib/libvmod_std/vmod.py b/lib/libvmod_std/vmod.py
index 7d751cb..a799dd8 100755
--- a/lib/libvmod_std/vmod.py
+++ b/lib/libvmod_std/vmod.py
@@ -141,7 +141,6 @@ def parse_enum(tq):
 			raise Exception("Duplicate Enum value '%s'" % i)
 		b[i] = True
 		s = s + i.strip() + '\\0'
-	s = s + '\\0'
 	return s
 
 #######################################################################



More information about the varnish-commit mailing list