Commit Diff

Commit:
f4a5cef1546205afab47f148edefabcf77c06d3b
From:
Omar Polo <op@omarpolo.com>
Date:
Message:
gotwebd: set chroot path in config_init() saves a bit fiddling around with ternary operators. sockets_run() will reset the chroot to the default since it calls again config_init(), but at that stage it's not important anymore. ok stsp@
Actions:
Patch | Tree
commit - 23c35dae297d2eecd13e17118a547761314a578d
commit + f4a5cef1546205afab47f148edefabcf77c06d3b
blob - a6a1722df89866f0ad5a9adb12cf2820cd9831bb
blob + b13f0b45ecb8c13c605728f061a869d69085a4e0
--- gotwebd/config.c
+++ gotwebd/config.c
@@ -47,6 +47,8 @@ config_init(struct gotwebd *env)
 {
 	struct privsep *ps = env->gotwebd_ps;
 	unsigned int what;
+
+	strlcpy(env->httpd_chroot, D_HTTPD_CHROOT, sizeof(env->httpd_chroot));
 
 	/* Global configuration. */
 	if (privsep_process == PROC_GOTWEBD)
blob - 3633c55f9eed4080f24d13cadb5b1fb99b89cf9c
blob + b9d0cee9431cab7cb4772f36a6f8519af76b3f6f
--- gotwebd/gotwebd.c
+++ gotwebd/gotwebd.c
@@ -213,8 +213,7 @@ main(int argc, char **argv)
 		ps->ps_title[proc_id] = title;
 
 	for (proc = 0; proc < nitems(procs); proc++)
-		procs[proc].p_chroot = strlen(env->httpd_chroot) ?
-		    env->httpd_chroot : D_HTTPD_CHROOT;
+		procs[proc].p_chroot = env->httpd_chroot;
 
 	/* only the gotwebd returns */
 	proc_init(ps, procs, nitems(procs), argc0, argv, proc_id);
@@ -251,8 +250,7 @@ main(int argc, char **argv)
 		err(1, "gmon.out");
 #endif
 
-	if (unveil(strlen(env->httpd_chroot) > 0 ? env->httpd_chroot :
-	    D_HTTPD_CHROOT, "rwc") == -1)
+	if (unveil(env->httpd_chroot, "rwc") == -1)
 		err(1, "unveil");
 
 	if (unveil(GOT_TMPDIR_STR, "rw") == -1)
blob - a20548a36dab44bcdc56af91c0847fc938ea5df6
blob + 5ef647901c4a642a50747f22cc3093323a84337f
--- gotwebd/parse.y
+++ gotwebd/parse.y
@@ -231,8 +231,7 @@ main		: PREFORK NUMBER {
 		| UNIX_SOCKET_NAME STRING {
 			n = snprintf(gotwebd->unix_socket_name,
 			    sizeof(gotwebd->unix_socket_name), "%s%s",
-			    strlen(gotwebd->httpd_chroot) ?
-			    gotwebd->httpd_chroot : D_HTTPD_CHROOT, $2);
+			    gotwebd->httpd_chroot, $2);
 			if (n < 0 ||
 			    (size_t)n >= sizeof(gotwebd->unix_socket_name)) {
 				yyerror("%s: unix_socket_name truncated",
@@ -364,8 +363,7 @@ serveropts1	: REPOS_PATH STRING {
 
 			n = snprintf(new_srv->unix_socket_name,
 			    sizeof(new_srv->unix_socket_name), "%s%s",
-			    strlen(gotwebd->httpd_chroot) ?
-			    gotwebd->httpd_chroot : D_HTTPD_CHROOT, $4);
+			    gotwebd->httpd_chroot, $4);
 			if (n < 0 ||
 			    (size_t)n >= sizeof(new_srv->unix_socket_name)) {
 				yyerror("%s: unix_socket_name truncated",