r4603 - trunk/varnish-cache/bin/varnishd

[Anders Nordby]

Hi,

I wonder if this fix should also be applied to http_copyh() as well,
which looks quite similar to http_copyheader(). It has the same assert.

Bye,
Anders.

On Wed, Mar 03, 2010 at 10:12:11AM +0100, phk at projects.linpro.no wrote:
> Author: phk
> Date: 2010-03-03 10:12:10 +0100 (Wed, 03 Mar 2010)
> New Revision: 4603
> 
> Modified:
>    trunk/varnish-cache/bin/varnishd/cache_http.c
> Log:
> Assert on the right HTTP structure.
> 
> Found by:	luc/tom28
> 
> 
> 
> Modified: trunk/varnish-cache/bin/varnishd/cache_http.c
> ===================================================================
> --- trunk/varnish-cache/bin/varnishd/cache_http.c	2010-03-01 11:30:43 UTC (rev 4602)
> +++ trunk/varnish-cache/bin/varnishd/cache_http.c	2010-03-03 09:12:10 UTC (rev 4603)
> @@ -644,7 +644,7 @@
>  
>  	CHECK_OBJ_NOTNULL(fm, HTTP_MAGIC);
>  	CHECK_OBJ_NOTNULL(to, HTTP_MAGIC);
> -	assert(n < to->shd);
> +	assert(n < fm->shd);
>  	Tcheck(fm->hd[n]);
>  	if (to->nhd < to->shd) {
>  		to->hd[to->nhd] = fm->hd[n];
> 
> _______________________________________________
> varnish-commit mailing list
> varnish-commit at projects.linpro.no
> http://projects.linpro.no/mailman/listinfo/varnish-commit


-- 
Anders.