r1789 - trunk/varnish-cache/bin/varnishd

des at projects.linpro.no des at projects.linpro.no
Mon Jul 30 16:22:01 CEST 2007


Author: des
Date: 2007-07-30 16:22:00 +0200 (Mon, 30 Jul 2007)
New Revision: 1789

Modified:
   trunk/varnish-cache/bin/varnishd/mgt_child.c
Log:
Bogons in previous commit.  Pass me the pointy hat...


Modified: trunk/varnish-cache/bin/varnishd/mgt_child.c
===================================================================
--- trunk/varnish-cache/bin/varnishd/mgt_child.c	2007-07-30 14:19:45 UTC (rev 1788)
+++ trunk/varnish-cache/bin/varnishd/mgt_child.c	2007-07-30 14:22:00 UTC (rev 1789)
@@ -197,8 +197,7 @@
 
 		/* Redirect stdin/out/err */
 		AZ(close(0));
-		i = open("/dev/null", O_RDONLY);
-		xxxassert(i == 0);
+		assert(open("/dev/null", O_RDONLY) == 0);
 		assert(dup2(child_fds[1], 1) == 1);
 		assert(dup2(child_fds[1], 2) == 2);
 		AZ(close(child_fds[0]));
@@ -213,7 +212,7 @@
 		signal(SIGTERM, SIG_DFL);
 		child_main();
 
-		exit (1);
+		exit(1);
 	}
 
 	fprintf(stderr, "start child pid %jd\n", (intmax_t)pid);
@@ -247,7 +246,7 @@
 	heritage.fds[1] = -1;
 	AZ(close(heritage.fds[2]));
 	heritage.fds[2] = -1;
-	child_pid = i;
+	child_pid = pid;
 	if (mgt_push_vcls_and_start(&u, &p)) {
 		fprintf(stderr, "Pushing vcls failed:\n%s\n", p);
 		free(p);




More information about the varnish-commit mailing list