runtime: don't call __go_alloc/__go_free in environment functions

    Reviewed-on: https://go-review.googlesource.com/33363

From-SVN: r242594

diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE
index ce86c8d6abf724c7a871fcb40651db1ab11e1b7f..6e29f40af134006d83f9143683d94c3c8e98cd51 100644 (file)
--- a/gcc/go/gofrontend/MERGE
+++ b/gcc/go/gofrontend/MERGE
@@ -1,4 +1,4 @@
-2ab785788691ad289f838a0b3a6bc9013d0fc337
+fc4ca600b2fc6de81fd3c4014542d6a50593db1a
 
 The first line of this file holds the git revision number of the last
 merge done from the gofrontend repository.

diff --git a/libgo/runtime/go-setenv.c b/libgo/runtime/go-setenv.c
index a75d7c412772362341cfc78d7a144a6c03ae8d78..81b1775d2c903a74ba5e3c0384cec40796b27789 100644 (file)
--- a/libgo/runtime/go-setenv.c
+++ b/libgo/runtime/go-setenv.c
@@ -9,10 +9,7 @@
 #include <stddef.h>
 #include <stdlib.h>
 
-#include "go-alloc.h"
 #include "runtime.h"
-#include "arch.h"
-#include "malloc.h"
 
 /* Set the C environment from Go.  This is called by syscall.Setenv.  */
 
@@ -25,7 +22,6 @@ setenv_c (String k, String v)
   unsigned char *kn;
   const byte *vs;
   unsigned char *vn;
-  intgo len;
 
   ks = k.str;
   if (ks == NULL)
@@ -39,25 +35,23 @@ setenv_c (String k, String v)
 
 #ifdef HAVE_SETENV
 
-  if (ks != NULL && ks[k.len] != 0)
+  if (ks[k.len] != 0)
     {
-      // Objects that are explicitly freed must be at least 16 bytes in size,
-      // so that they are not allocated using tiny alloc.
-      len = k.len + 1;
-      if (len < TinySize)
-       len = TinySize;
-      kn = __go_alloc (len);
+      kn = malloc (k.len + 1);
+      if (kn == NULL)
+       runtime_throw ("out of malloc memory");
       __builtin_memcpy (kn, ks, k.len);
+      kn[k.len] = '\0';
       ks = kn;
     }
 
-  if (vs != NULL && vs[v.len] != 0)
+  if (vs[v.len] != 0)
     {
-      len = v.len + 1;
-      if (len < TinySize)
-       len = TinySize;
-      vn = __go_alloc (len);
+      vn = malloc (v.len + 1);
+      if (vn == NULL)
+       runtime_throw ("out of malloc memory");
       __builtin_memcpy (vn, vs, v.len);
+      vn[v.len] = '\0';
       vs = vn;
     }
 
@@ -66,19 +60,20 @@ setenv_c (String k, String v)
 #else /* !defined(HAVE_SETENV) */
 
   len = k.len + v.len + 2;
-  if (len < TinySize)
-    len = TinySize;
-  kn = __go_alloc (len);
+  kn = malloc (len);
+  if (kn == NULL)
+    runtime_throw ("out of malloc memory");
   __builtin_memcpy (kn, ks, k.len);
   kn[k.len] = '=';
   __builtin_memcpy (kn + k.len + 1, vs, v.len);
   kn[k.len + v.len + 1] = '\0';
   putenv ((char *) kn);
+  kn = NULL; /* putenv takes ownership of the string.  */
 
 #endif /* !defined(HAVE_SETENV) */
 
   if (kn != NULL)
-    __go_free (kn);
+    free (kn);
   if (vn != NULL)
-    __go_free (vn);
+    free (vn);
 }

diff --git a/libgo/runtime/go-unsetenv.c b/libgo/runtime/go-unsetenv.c
index 409436a0d3f7dd913b704bb2e163d43322769f1f..21359975f2b3e9053496482ef853ddf4192bab0b 100644 (file)
--- a/libgo/runtime/go-unsetenv.c
+++ b/libgo/runtime/go-unsetenv.c
@@ -9,10 +9,7 @@
 #include <stddef.h>
 #include <stdlib.h>
 
-#include "go-alloc.h"
 #include "runtime.h"
-#include "arch.h"
-#include "malloc.h"
 
 /* Unset an environment variable from Go.  This is called by
    syscall.Unsetenv.  */
@@ -24,7 +21,6 @@ unsetenv_c (String k)
 {
   const byte *ks;
   unsigned char *kn;
-  intgo len;
 
   ks = k.str;
   if (ks == NULL)
@@ -33,14 +29,11 @@ unsetenv_c (String k)
 
 #ifdef HAVE_UNSETENV
 
-  if (ks != NULL && ks[k.len] != 0)
+  if (ks[k.len] != 0)
     {
-      // Objects that are explicitly freed must be at least 16 bytes in size,
-      // so that they are not allocated using tiny alloc.
-      len = k.len + 1;
-      if (len < TinySize)
-       len = TinySize;
-      kn = __go_alloc (len);
+      kn = malloc (k.len + 1);
+      if (kn == NULL)
+       runtime_throw ("out of malloc memory");
       __builtin_memcpy (kn, ks, k.len);
       ks = kn;
     }
@@ -50,5 +43,5 @@ unsetenv_c (String k)
 #endif /* !defined(HAVE_UNSETENV) */
 
   if (kn != NULL)
-    __go_free (kn);
+    free (kn);
 }