[master] ae69070 dont assert on a failing rmdir

Nils Goroll nils.goroll at uplex.de
Wed Apr 20 15:08:05 CEST 2016


commit ae69070c3aa5073808c5ba8c94009cfb50da789a
Author: Nils Goroll <nils.goroll at uplex.de>
Date:   Wed Apr 20 15:06:44 2016 +0200

    dont assert on a failing rmdir
    
    We may not have permission to unlink vp.dir
    
    $ /etc/init.d/varnish_master restart
    Stopping Varnish Cache:                                    [FAILED]
    Starting Varnish Cache: Assert error in mgt_VccCompile(), mgt/mgt_vcc.c line 302:
      Condition((rmdir(vp.dir)) == 0) not true.
      errno = 13 (Permission denied)

diff --git a/bin/varnishd/mgt/mgt_vcc.c b/bin/varnishd/mgt/mgt_vcc.c
index c0f8ed2..9a094fb 100644
--- a/bin/varnishd/mgt/mgt_vcc.c
+++ b/bin/varnishd/mgt/mgt_vcc.c
@@ -299,7 +299,7 @@ mgt_VccCompile(struct cli *cli, const char *vclname, const char *vclsrc,
 	if (status || C_flag) {
 		(void)unlink(vp.libfile);
 		free(vp.libfile);
-		AZ(rmdir(vp.dir));
+		(void)rmdir(vp.dir);
 		free(vp.dir);
 		if (status) {
 			VCLI_Out(cli, "VCL compilation failed");



More information about the varnish-commit mailing list