[master] 64370e2 Use closefd() throughout

Poul-Henning Kamp phk at FreeBSD.org
Sat Jan 14 20:49:05 CET 2017


commit 64370e2f088731b1a536354875cc770abf9e8140
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Sat Jan 14 19:28:16 2017 +0000

    Use closefd() throughout

diff --git a/bin/varnishd/cache/cache_acceptor.c b/bin/varnishd/cache/cache_acceptor.c
index 754d1b0..cb907a9 100644
--- a/bin/varnishd/cache/cache_acceptor.c
+++ b/bin/varnishd/cache/cache_acceptor.c
@@ -305,7 +305,7 @@ vca_make_session(struct worker *wrk, void *arg)
 	CAST_OBJ_NOTNULL(wa, arg, WRK_ACCEPT_MAGIC);
 
 	if (VTCP_blocking(wa->acceptsock)) {
-		AZ(close(wa->acceptsock));
+		closefd(&wa->acceptsock);
 		wrk->stats->sess_drop++;	// XXX Better counter ?
 		WS_Release(wrk->aws, 0);
 		return;
@@ -322,7 +322,7 @@ vca_make_session(struct worker *wrk, void *arg)
 		 */
 		vca_pace_bad();
 		(void)VTCP_nonblocking(wa->acceptsock);
-		AZ(close(wa->acceptsock));
+		closefd(&wa->acceptsock);
 		wrk->stats->sess_drop++;
 		WS_Release(wrk->aws, 0);
 		return;
diff --git a/bin/varnishd/cache/cache_vrt_vmod.c b/bin/varnishd/cache/cache_vrt_vmod.c
index a938568..f64cb33 100644
--- a/bin/varnishd/cache/cache_vrt_vmod.c
+++ b/bin/varnishd/cache/cache_vrt_vmod.c
@@ -86,7 +86,7 @@ vrt_vmod_backup_copy(VRT_CTX, const char *nm, const char *fm, const char *to)
 		VSB_printf(ctx->msg, "Opening vmod %s from %s: %s\n",
 		    nm, fm, strerror(errno));
 		AZ(unlink(to));
-		AZ(close(fo));
+		closefd(&fo);
 		return (1);
 	}
 	while (1) {
@@ -101,8 +101,8 @@ vrt_vmod_backup_copy(VRT_CTX, const char *nm, const char *fm, const char *to)
 			break;
 		}
 	}
-	AZ(close(fi));
-	AZ(close(fo));
+	closefd(&fi);
+	closefd(&fo);
 	return(ret);
 }
 
diff --git a/bin/varnishd/mgt/mgt_acceptor.c b/bin/varnishd/mgt/mgt_acceptor.c
index eabad1a..3b0985f 100644
--- a/bin/varnishd/mgt/mgt_acceptor.c
+++ b/bin/varnishd/mgt/mgt_acceptor.c
@@ -59,7 +59,7 @@ mac_opensocket(struct listen_sock *ls)
 	CHECK_OBJ_NOTNULL(ls, LISTEN_SOCK_MAGIC);
 	if (ls->sock > 0) {
 		MCH_Fd_Inherit(ls->sock, NULL);
-		AZ(close(ls->sock));
+		closefd(&ls->sock);
 	}
 	ls->sock = VTCP_bind(ls->addr, NULL);
 	fail = errno;
diff --git a/bin/varnishd/mgt/mgt_cli.c b/bin/varnishd/mgt/mgt_cli.c
index 2f9ca91..eeeef21 100644
--- a/bin/varnishd/mgt/mgt_cli.c
+++ b/bin/varnishd/mgt/mgt_cli.c
@@ -282,7 +282,7 @@ mcf_auth(struct cli *cli, const char *const *av, void *priv)
 	VJ_master(JAIL_MASTER_LOW);
 	MCH_TrackHighFd(fd);
 	VCLI_AuthResponse(fd, cli->challenge, buf);
-	AZ(close(fd));
+	closefd(&fd);
 	if (strcasecmp(buf, av[2])) {
 		MGT_Complain(C_SECURITY,
 		    "CLI Authentication failure from %s", cli->ident);
@@ -518,7 +518,7 @@ mgt_cli_secret(const char *S_arg)
 		fprintf(stderr, "Can not read secret-file \"%s\"\n", S_arg);
 		exit(2);
 	}
-	AZ(close(fd));
+	closefd(&fd);
 	secret_file = S_arg;
 }
 
diff --git a/bin/varnishd/mgt/mgt_jail.c b/bin/varnishd/mgt/mgt_jail.c
index 515d56d..85cd840 100644
--- a/bin/varnishd/mgt/mgt_jail.c
+++ b/bin/varnishd/mgt/mgt_jail.c
@@ -156,7 +156,7 @@ VJ_make_workdir(const char *dname)
 		ARGV_ERR("Cannot create test-file in %s (%s)\n"
 		    "Check permissions (or delete old directory)\n",
 		    dname, strerror(errno));
-	AZ(close(fd));
+	closefd(&fd);
 	AZ(unlink("_.testfile"));
 	VJ_master(JAIL_MASTER_LOW);
 	return (0);
diff --git a/bin/varnishd/mgt/mgt_main.c b/bin/varnishd/mgt/mgt_main.c
index fd636c0..eb5cbb8 100644
--- a/bin/varnishd/mgt/mgt_main.c
+++ b/bin/varnishd/mgt/mgt_main.c
@@ -219,7 +219,7 @@ make_secret(const char *dirname)
 		AZ(VRND_RandomCrypto(&b, 1));
 		assert(1 == write(fdo, &b, 1));
 	}
-	AZ(close(fdo));
+	closefd(&fdo);
 	VJ_master(JAIL_MASTER_LOW);
 	AZ(atexit(mgt_secret_atexit));
 	return (fn);
@@ -361,7 +361,7 @@ mgt_eric(void)
 		fprintf(stderr, "Fork() failed: %s\n", strerror(errno));
 		exit(-1);
 	case 0:
-		AZ(close(eric_pipes[0]));
+		closefd(&eric_pipes[0]);
 		assert(setsid() > 1);
 
 		VFIL_null_fd(STDIN_FILENO);
@@ -369,7 +369,7 @@ mgt_eric(void)
 	default:
 		break;
 	}
-	AZ(close(eric_pipes[1]));
+	closefd(&eric_pipes[1]);
 	sz = read(eric_pipes[0], &u, sizeof u);
 	if (sz == sizeof u && u == ERIC_MAGIC)
 		exit(0);
@@ -391,7 +391,7 @@ mgt_eric_im_done(int eric_fd, unsigned u)
 	VFIL_null_fd(STDERR_FILENO);
 
 	assert(write(eric_fd, &u, sizeof u) == sizeof u);
-	AZ(close(eric_fd));
+	closefd(&eric_fd);
 }
 
 /*--------------------------------------------------------------------*/
@@ -681,7 +681,7 @@ main(int argc, char * const *argv)
 		if (o < 0)
 			ARGV_ERR("Cannot open -S file (%s): %s\n",
 			    S_arg, strerror(errno));
-		AZ(close(o));
+		closefd(&o);
 		VJ_master(JAIL_MASTER_LOW);
 	}
 
diff --git a/bin/varnishd/mgt/mgt_param_tcp.c b/bin/varnishd/mgt/mgt_param_tcp.c
index 7744e1a..9567d6f 100644
--- a/bin/varnishd/mgt/mgt_param_tcp.c
+++ b/bin/varnishd/mgt/mgt_param_tcp.c
@@ -75,7 +75,7 @@ tcp_keep_probes(void)
 	tcp_probe(s, TCP_KEEPIDLE, "tcp_keepalive_time",	600);
 	tcp_probe(s, TCP_KEEPCNT, "tcp_keepalive_probes",	5);
 	tcp_probe(s, TCP_KEEPINTVL, "tcp_keepalive_intvl",	5);
-	AZ(close(s));
+	closefd(&s);
 }
 #endif
 
diff --git a/bin/varnishd/mgt/mgt_shmem.c b/bin/varnishd/mgt/mgt_shmem.c
index 7ddcefc..e41018d 100644
--- a/bin/varnishd/mgt/mgt_shmem.c
+++ b/bin/varnishd/mgt/mgt_shmem.c
@@ -218,7 +218,7 @@ mgt_SHM_Create(void)
 	    MAP_HASSEMAPHORE | MAP_NOSYNC | MAP_SHARED,
 	    vsm_fd, 0);
 
-	AZ(close(vsm_fd));
+	closefd(&vsm_fd);
 
 	if (p == MAP_FAILED) {
 		MGT_Complain(C_ERR, "Mmap error %s: %s",
diff --git a/bin/varnishd/mgt/mgt_vcc.c b/bin/varnishd/mgt/mgt_vcc.c
index 39e2803..863a49c 100644
--- a/bin/varnishd/mgt/mgt_vcc.c
+++ b/bin/varnishd/mgt/mgt_vcc.c
@@ -125,7 +125,7 @@ run_vcc(void *priv)
 		fprintf(stderr, "VCC cannot write %s", vp->csrcfile);
 		exit(2);
 	}
-	AZ(close(fd));
+	closefd(&fd);
 	VSB_destroy(&csrc);
 	exit(0);
 }
@@ -216,7 +216,7 @@ mgt_vcc_touchfile(const char *fn, struct vsb *sb)
 		if (geteuid() == 0)
 			VSB_printf(sb, "Failed to change owner on %s: %s\n",
 			    fn, strerror(errno));
-	AZ(close(i));
+	closefd(&i);
 	return (0);
 }
 
diff --git a/bin/varnishd/waiter/cache_waiter_epoll.c b/bin/varnishd/waiter/cache_waiter_epoll.c
index f19b20b..0a59592 100644
--- a/bin/varnishd/waiter/cache_waiter_epoll.c
+++ b/bin/varnishd/waiter/cache_waiter_epoll.c
@@ -146,9 +146,9 @@ vwe_thread(void *priv)
 		if (vwe->nwaited == 0 && vwe->die)
 			break;
 	}
-	AZ(close(vwe->pipe[0]));
-	AZ(close(vwe->pipe[1]));
-	AZ(close(vwe->epfd));
+	closefd(&vwe->pipe[0]);
+	closefd(&vwe->pipe[1]);
+	closefd(&vwe->epfd);
 	return (NULL);
 }
 
diff --git a/bin/varnishd/waiter/cache_waiter_kqueue.c b/bin/varnishd/waiter/cache_waiter_kqueue.c
index 4832f49..e757e40 100644
--- a/bin/varnishd/waiter/cache_waiter_kqueue.c
+++ b/bin/varnishd/waiter/cache_waiter_kqueue.c
@@ -128,9 +128,9 @@ vwk_thread(void *priv)
 		if (vwk->nwaited == 0 && vwk->die)
 			break;
 	}
-	AZ(close(vwk->pipe[0]));
-	AZ(close(vwk->pipe[1]));
-	AZ(close(vwk->kq));
+	closefd(&vwk->pipe[0]);
+	closefd(&vwk->pipe[1]);
+	closefd(&vwk->kq);
 	return(NULL);
 }
 
diff --git a/bin/varnishd/waiter/cache_waiter_poll.c b/bin/varnishd/waiter/cache_waiter_poll.c
index 282d7a8..19fde5d 100644
--- a/bin/varnishd/waiter/cache_waiter_poll.c
+++ b/bin/varnishd/waiter/cache_waiter_poll.c
@@ -262,8 +262,8 @@ vwp_fini(struct waiter *w)
 	// XXX: set write pipe blocking
 	assert(write(vwp->pipes[1], &vp, sizeof vp) == sizeof vp);
 	AZ(pthread_join(vwp->thread, &vp));
-	AZ(close(vwp->pipes[0]));
-	AZ(close(vwp->pipes[1]));
+	closefd(&vwp->pipes[0]);
+	closefd(&vwp->pipes[1]);
 	free(vwp->pollfd);
 	free(vwp->idx);
 }
diff --git a/bin/varnishd/waiter/cache_waiter_ports.c b/bin/varnishd/waiter/cache_waiter_ports.c
index 5b11be3..a7b2e47 100644
--- a/bin/varnishd/waiter/cache_waiter_ports.c
+++ b/bin/varnishd/waiter/cache_waiter_ports.c
@@ -256,7 +256,7 @@ vws_fini(struct waiter *w)
 
 	CAST_OBJ_NOTNULL(vws, w->priv, VWS_MAGIC);
 	vws->die = 1;
-	AZ(close(vws->dport));
+	closefd(&vws->dport);
 	AZ(pthread_join(vws->thread, &vp));
 }
 



More information about the varnish-commit mailing list