r3183 - trunk/varnish-cache/bin/varnishd

phk at projects.linpro.no phk at projects.linpro.no
Fri Sep 12 11:39:35 CEST 2008


Author: phk
Date: 2008-09-12 11:39:35 +0200 (Fri, 12 Sep 2008)
New Revision: 3183

Modified:
   trunk/varnish-cache/bin/varnishd/cache_vrt.c
Log:
Don't allocate space for a %u and then stick a %d in it.

Tripped by:	anordby



Modified: trunk/varnish-cache/bin/varnishd/cache_vrt.c
===================================================================
--- trunk/varnish-cache/bin/varnishd/cache_vrt.c	2008-09-12 08:51:02 UTC (rev 3182)
+++ trunk/varnish-cache/bin/varnishd/cache_vrt.c	2008-09-12 09:39:35 UTC (rev 3183)
@@ -480,7 +480,7 @@
 
 	size = snprintf(NULL, 0, "%u", sp->xid) + 1;
 	AN(p = WS_Alloc(sp->http->ws, size));
-	assert(snprintf(p, size, "%d", sp->xid) < size);
+	assert(snprintf(p, size, "%u", sp->xid) < size);
 	return (p);
 }
 




More information about the varnish-commit mailing list