[master] c81eced Dont tell worker "0auto" if we mean "0cold"

Poul-Henning Kamp phk at FreeBSD.org
Mon Nov 27 12:30:13 UTC 2017


commit c81eced9eaa83754c87027b88295ee9cc685f25e
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Mon Nov 27 11:51:00 2017 +0000

    Dont tell worker "0auto" if we mean "0cold"

diff --git a/bin/varnishd/mgt/mgt_vcl.c b/bin/varnishd/mgt/mgt_vcl.c
index d2428b3..224082b 100644
--- a/bin/varnishd/mgt/mgt_vcl.c
+++ b/bin/varnishd/mgt/mgt_vcl.c
@@ -430,7 +430,7 @@ mgt_vcl_setstate(struct cli *cli, struct vclprog *vp, const char *vs)
 	}
 
 	i = mgt_cli_askchild(&status, &p, "vcl.state %s %d%s\n",
-	    vp->name, warm, vp->state);
+	    vp->name, warm, vs);
 	if (i && cli != NULL) {
 		VCLI_SetResult(cli, status);
 		VCLI_Out(cli, "%s", p);
@@ -750,6 +750,7 @@ mcf_vcl_discard(struct cli *cli, const char * const *av, void *priv)
 		(void)mgt_vcl_setstate(cli, vp, VCL_STATE_COLD);
 	if (MCH_Running()) {
 		/* XXX If this fails the child is crashing, figure that later */
+		assert(!vp->warm);
 		(void)mgt_cli_askchild(&status, &p, "vcl.discard %s\n", av[2]);
 		free(p);
 	}


More information about the varnish-commit mailing list