Changeset 3706 for branches/libc-0.6/src

Timestamp:

Mar 16, 2011, 10:12:11 PM (14 years ago)

Author:

bird

Message:

0.6: Backported r3705: iconv.c: Use malloc instead of alloca for larger strings so we don't blow up the stack. Fixes #183.

File:

• branches/libc-0.6/src/emx/src/lib/locale/iconv.c (modified) (4 diffs)

branches/libc-0.6/src/emx/src/lib/locale/iconv.c

@@ -158 +158 @@
   size_t    nonid;
   UniChar  *ucs;
+  UniChar  *pucsFree = NULL;
   UniChar  *orig_ucs;
   size_t    retval = 0;
@@ -185 +186 @@
     }
 
-  sl =  *in_left;
-  ucs = (UniChar *) alloca (sl * sizeof (UniChar));
+  sl = *in_left;
+  if (sl <= 1024)
+    ucs = (UniChar *) alloca (sl * sizeof (UniChar));
+  else
+    {
+      pucsFree = ucs = (UniChar *) malloc (sl * sizeof (UniChar));
+      if (!ucs)
+        {
+          errno = ENOMEM;
+          return -1;
+        }
+    }
   orig_ucs = ucs;
 
@@ -220 +231 @@
     }
 #endif
+  if (pucsFree)
+      free(pucsFree);
   FS_RESTORE();
   return 0;
@@ -240 +253 @@
       break;
   }
+  if (pucsFree)
+      free(pucsFree);
   FS_RESTORE();
   return (size_t)(-1);