From 1feaf2a618ee1c4771fee80ced7acf31fe40fdae Mon Sep 17 00:00:00 2001
From: Omar Polo <op@omarpolo.com>
Date: Sat, 15 May 2021 10:31:43 +0000
Subject: [PATCH] use the correct document root

pass the correct loc_off to the executor, so the various variables
that depends on the matched location (like DOCUMENT_ROOT) are computed
correctly.
---
 ex.c     |  2 +-
 gmid.h   |  3 ++-
 server.c | 13 ++++++++++---
 3 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/ex.c b/ex.c
index 0dcad61..39eedec 100644
--- a/ex.c
+++ b/ex.c
@@ -290,7 +290,7 @@ handle_imsg_cgi_req(struct imsgbuf *ibuf, struct imsg *imsg, size_t datalen)
 	if ((h = host_nth(req.host_off)) == NULL)
 		abort();
 
-	if ((l = loc_nth(h, req.host_off)) == NULL)
+	if ((l = loc_nth(h, req.loc_off)) == NULL)
 		abort();
 
 	fd = launch_cgi(&iri, &req, h, l);
diff --git a/gmid.h b/gmid.h
index e579471..2fbec06 100644
--- a/gmid.h
+++ b/gmid.h
@@ -245,6 +245,7 @@ struct client {
 
 	struct sockaddr_storage	 addr;
 	struct vhost	*host;	/* host they're talking to */
+	size_t		 loc;	/* location matched */
 };
 
 extern struct client clients[MAX_USERS];
@@ -342,7 +343,7 @@ const char	*vhost_index(struct vhost*, const char*);
 int		 vhost_auto_index(struct vhost*, const char*);
 int		 vhost_block_return(struct vhost*, const char*, int*, const char**);
 int		 vhost_fastcgi(struct vhost*, const char*);
-int		 vhost_dirfd(struct vhost*, const char*);
+int		 vhost_dirfd(struct vhost*, const char*, size_t*);
 int		 vhost_strip(struct vhost*, const char*);
 X509_STORE	*vhost_require_ca(struct vhost*, const char*);
 int		 vhost_disable_log(struct vhost*, const char*);
diff --git a/server.c b/server.c
index b7a71f4..514782b 100644
--- a/server.c
+++ b/server.c
@@ -222,20 +222,25 @@ vhost_fastcgi(struct vhost *v, const char *path)
 }
 
 int
-vhost_dirfd(struct vhost *v, const char *path)
+vhost_dirfd(struct vhost *v, const char *path, size_t *retloc)
 {
 	struct location *loc;
+	size_t		 l = 0;
 
 	if (v == NULL || path == NULL)
 		return -1;
 
 	loc = TAILQ_FIRST(&v->locations);
 	while ((loc = TAILQ_NEXT(loc, locations)) != NULL) {
+		l++;
 		if (loc->dirfd != -1)
-			if (matches(loc->match, path))
+			if (matches(loc->match, path)) {
+				*retloc = l;
 				return loc->dirfd;
+			}
 	}
 
+	*retloc = 0;
 	loc = TAILQ_FIRST(&v->locations);
 	return loc->dirfd;
 }
@@ -322,7 +327,7 @@ check_path(struct client *c, const char *path, int *fd)
 	if (*p == '\0')
 		p = ".";
 
-	dirfd = vhost_dirfd(c->host, path);
+	dirfd = vhost_dirfd(c->host, path, &c->loc);
 	log_debug(c, "check_path: strip=%d path=%s original=%s",
 	    strip, p, path);
 	flags = O_RDONLY | O_NOFOLLOW;
@@ -683,6 +688,7 @@ handle_open_conn(int fd, short ev, void *d)
 		return;
 
 	if (c->host->entrypoint != NULL) {
+		c->loc = 0;
 		start_cgi(c->host->entrypoint, c->iri.path, c);
 		return;
 	}
@@ -802,6 +808,7 @@ start_cgi(const char *spath, const char *relpath, struct client *c)
 	req.notafter = tls_peer_cert_notafter(c->ctx);
 
 	req.host_off = host_nth(c->host);
+	req.loc_off = c->loc;
 
 	imsg_compose(&exibuf, IMSG_CGI_REQ, c->id, 0, -1, &req, sizeof(req));
 	imsg_flush(&exibuf);