r3711 - branches/2.0/varnish-cache/bin/varnishd

tfheen at projects.linpro.no tfheen at projects.linpro.no
Mon Feb 9 14:18:50 CET 2009


Author: tfheen
Date: 2009-02-09 14:18:50 +0100 (Mon, 09 Feb 2009)
New Revision: 3711

Modified:
   branches/2.0/varnish-cache/bin/varnishd/cache_ban.c
Log:
Merge r3523: A couple of FlexeLint nits



Modified: branches/2.0/varnish-cache/bin/varnishd/cache_ban.c
===================================================================
--- branches/2.0/varnish-cache/bin/varnishd/cache_ban.c	2009-02-09 13:15:50 UTC (rev 3710)
+++ branches/2.0/varnish-cache/bin/varnishd/cache_ban.c	2009-02-09 13:18:50 UTC (rev 3711)
@@ -242,7 +242,7 @@
 
 	(void)o;
 	CHECK_OBJ_NOTNULL(sp, SESS_MAGIC);
-	http_GetHdr(sp->http, bt->src, &s);
+	(void)http_GetHdr(sp->http, bt->src, &s);
 	return (ban_cond_str(bt, s));
 }
 
@@ -254,7 +254,7 @@
 
 	(void)sp;
 	CHECK_OBJ_NOTNULL(o, OBJECT_MAGIC);
-	http_GetHdr(o->http, bt->src, &s);
+	(void)http_GetHdr(o->http, bt->src, &s);
 	return (ban_cond_str(bt, s));
 }
 
@@ -288,9 +288,10 @@
 	int l;
 
 	l = strlen(a1);
+	assert(l < 127);
 	bt->src = malloc(l + 3);
 	XXXAN(bt->src);
-	bt->src[0] = l + 1;
+	bt->src[0] = (char)l + 1;
 	memcpy(bt->src + 1, a1, l);
 	bt->src[l + 1] = ':';
 	bt->src[l + 2] = '\0';



More information about the varnish-commit mailing list