about summary refs log tree commit diff
path: root/stdlib/setenv.c
diff options
context:
space:
mode:
authorJoe Simmons-Talbott <josimmon@redhat.com>2023-06-30 14:31:45 +0000
committerJoe Simmons-Talbott <josimmon@redhat.com>2023-06-30 14:31:45 +0000
commit9401024e5e6be0e1c3870e185daae865cd4501f4 (patch)
tree80b333a6cf8e36e36a428b9b423b35904e4ddc2b /stdlib/setenv.c
parent9555be54ef94e5d017ce4235e4f7c4e16662e17e (diff)
downloadglibc-9401024e5e6be0e1c3870e185daae865cd4501f4.tar.gz
glibc-9401024e5e6be0e1c3870e185daae865cd4501f4.tar.xz
glibc-9401024e5e6be0e1c3870e185daae865cd4501f4.zip
setenv.c: Get rid of alloca.
Use malloc rather than alloca to avoid potential stack overflow.

Reviewed-by: Adhemerval Zanella  <adhemerval.zanella@linaro.org>
Diffstat (limited to 'stdlib/setenv.c')
-rw-r--r--stdlib/setenv.c40
1 files changed, 6 insertions, 34 deletions
diff --git a/stdlib/setenv.c b/stdlib/setenv.c
index ba5257d3bf..cc71287fcc 100644
--- a/stdlib/setenv.c
+++ b/stdlib/setenv.c
@@ -182,18 +182,11 @@ __add_to_environ (const char *name, const char *value, const char *combined,
 	{
 	  const size_t varlen = namelen + 1 + vallen;
 #ifdef USE_TSEARCH
-	  char *new_value;
-	  int use_alloca = __libc_use_alloca (varlen);
-	  if (__builtin_expect (use_alloca, 1))
-	    new_value = (char *) alloca (varlen);
-	  else
+	  char *new_value = malloc (varlen);
+	  if (new_value == NULL)
 	    {
-	      new_value = malloc (varlen);
-	      if (new_value == NULL)
-		{
-		  UNLOCK;
-		  return -1;
-		}
+	      UNLOCK;
+	      return -1;
 	    }
 # ifdef _LIBC
 	  __mempcpy (__mempcpy (__mempcpy (new_value, name, namelen), "=", 1),
@@ -209,35 +202,14 @@ __add_to_environ (const char *name, const char *value, const char *combined,
 #endif
 	    {
 #ifdef USE_TSEARCH
-	      if (__glibc_unlikely (! use_alloca))
-		np = new_value;
-	      else
+	      np = new_value;
 #endif
-		{
-		  np = malloc (varlen);
-		  if (__glibc_unlikely (np == NULL))
-		    {
-		      UNLOCK;
-		      return -1;
-		    }
-
-#ifdef USE_TSEARCH
-		  memcpy (np, new_value, varlen);
-#else
-		  memcpy (np, name, namelen);
-		  np[namelen] = '=';
-		  memcpy (&np[namelen + 1], value, vallen);
-#endif
-		}
 	      /* And remember the value.  */
 	      STORE_VALUE (np);
 	    }
 #ifdef USE_TSEARCH
 	  else
-	    {
-	      if (__glibc_unlikely (! use_alloca))
-		free (new_value);
-	    }
+	    free (new_value);
 #endif
 	}