git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v2 1/4] compat/setenv.c: update errno when erroring out

From
Erik Faye-Lund <kusmabite@gmail.com>
Date
Dec 14, 2011, 14:07 UTC
Message-ID
<1323871631-2872-2-git-send-email-kusmabite@gmail.com>
In-Reply-To
<1323871631-2872-1-git-send-email-kusmabite@gmail.com>

Previously, gitsetenv didn't update errno as it should when erroring out. Fix this.

Signed-off-by: Erik Faye-Lund <kusmabite@gmail.com>
---
 compat/setenv.c |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/compat/setenv.c b/compat/setenv.c
index 3a22ea7..89947b7 100644
--- a/compat/setenv.c
+++ b/compat/setenv.c
@@ -6,7 +6,10 @@ int gitsetenv(const char *name, const char *value, int replace)
 	size_t namelen, valuelen;
 	char *envstr;
 
-	if (!name || !value) return -1;
+	if (!name || !value) {
+		errno = EINVAL;
+		return -1;
+	}
 	if (!replace) {
 		char *oldval = NULL;
 		oldval = getenv(name);
@@ -16,7 +19,10 @@ int gitsetenv(const char *name, const char *value, int replace)
 	namelen = strlen(name);
 	valuelen = strlen(value);
 	envstr = malloc((namelen + valuelen + 2));
-	if (!envstr) return -1;
+	if (!envstr) {
+		errno = ENOMEM;
+		return -1;
+	}
 
 	memcpy(envstr, name, namelen);
 	envstr[namelen] = '=';
-- 
1.7.7.1.msysgit.0.272.g9e47e
Previous: Erik Faye-LundNext: Erik Faye-Lund
Message 2 of 9 in “setenv/putenv errors”
  1. 0/4 setenv/putenv errorsErik Faye-Lund, Dec 14, 2011
  2. 1/4 compat/setenv.c: update errno when erroring outErik Faye-Lund, Dec 14, 2011
  3. 2/4 compat/setenv.c: error if name contains '='Erik Faye-Lund, Dec 14, 2011
  4. 3/4 wrapper: supply xsetenv and xputenvErik Faye-Lund, Dec 14, 2011
  5. Junio C HamanoDec 15, 2011
  6. Erik Faye-LundDec 15, 2011
  7. 4/4 use wrapper for unchecked setenv/putenv callsErik Faye-Lund, Dec 14, 2011
  8. Jeff KingDec 14, 2011
  9. Erik Faye-LundDec 15, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.