From e14ef76efd40ce8fb97bc462e7de3826dd90e847 Mon Sep 17 00:00:00 2001
From: Pete Moore <pmoore@mozilla.com>
Date: Thu, 18 Jan 2024 14:49:02 +0100
Subject: [PATCH] Issue #483 - fix getenv call to only specify variable name

Tup was previously calling getenv("FOO=bar") to retrieve envvar FOO
inside tup_db_findenv. This worked on macOS, but not on Linux. Updated
to call getenv(varname) instead (e.g. getenv("FOO")). Fixes #483
---
 src/tup/db.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/tup/db.c b/src/tup/db.c
index 5ce8a818a..1bfd5c931 100644
--- a/src/tup/db.c
+++ b/src/tup/db.c
@@ -5220,7 +5220,11 @@ int tup_db_findenv(const char *var, int varlen, struct var_entry **ret)
 		newtent = node_insert(env_dtent(), var, varlen, NULL, 0, NULL, 0, TUP_NODE_VAR, INVALID_MTIME, -1);
 		if(!newtent)
 			return -1;
-		newenv = getenv(var);
+		char* varname = strndup(var, varlen);
+		if(varname == NULL)
+			return -1;
+		newenv = getenv(varname);
+		free(varname);
 		if(newenv)
 			newenvlen = strlen(newenv);
 		ve = envdb_set(var, varlen, newenv, newenvlen, newtent, 1);