From 26b03c9ffe4d0037c3139297ac8e2689b489bbb1 Mon Sep 17 00:00:00 2001
From: Vivek Dasmohapatra <vivek@collabora.co.uk>
Date: Fri, 5 Feb 2010 13:56:44 +0000
Subject: [PATCH] Use xmalloc/xfree instead of alloc to allocate large tempporary charset structs.


diff --git a/src/charset.c b/src/charset.c
index 9e8ff1d..48339c2 100644
--- a/src/charset.c
+++ b/src/charset.c
@@ -526,7 +526,8 @@ load_charset_map_from_file (charset, mapfile, control_flag)
     error ("Failure in loading charset map: %S", SDATA (mapfile));
 
   head = entries = ((struct charset_map_entries *)
-		    xmalloc (sizeof (struct charset_map_entries)));
+                    xmalloc (sizeof (struct charset_map_entries)));
+
   n_entries = 0;
   eof = 0;
   while (1)
@@ -550,9 +551,10 @@ load_charset_map_from_file (charset, mapfile, control_flag)
       if (n_entries > 0 && (n_entries % 0x10000) == 0)
 	{
 	  entries->next = ((struct charset_map_entries *)
-			   alloca (sizeof (struct charset_map_entries)));
+                           xmalloc (sizeof (struct charset_map_entries)));
 	  entries = entries->next;
 	}
+
       idx = n_entries % 0x10000;
       entries->entry[idx].from = from;
       entries->entry[idx].to = to;
@@ -563,7 +565,12 @@ load_charset_map_from_file (charset, mapfile, control_flag)
   close (fd);
 
   load_charset_map (charset, head, n_entries, control_flag);
-  xfree (head);
+
+  for (entries = head; entries; entries = head)
+    {
+      head = entries->next; /* move the head of the list on */
+      xfree (entries);      /* free the old head            */
+    }
 }
 
 static void
@@ -586,7 +593,8 @@ load_charset_map_from_vector (charset, vec, control_flag)
     }
 
   head = entries = ((struct charset_map_entries *)
-		    alloca (sizeof (struct charset_map_entries)));
+                    xmalloc (sizeof (struct charset_map_entries)));
+
   n_entries = 0;
   for (i = 0; i < len; i += 2)
     {
@@ -620,7 +628,7 @@ load_charset_map_from_vector (charset, vec, control_flag)
       if (n_entries > 0 && (n_entries % 0x10000) == 0)
 	{
 	  entries->next = ((struct charset_map_entries *)
-			   alloca (sizeof (struct charset_map_entries)));
+                           xmalloc (sizeof (struct charset_map_entries), 1));
 	  entries = entries->next;
 	}
       idx = n_entries % 0x10000;
@@ -631,6 +639,11 @@ load_charset_map_from_vector (charset, vec, control_flag)
     }
 
   load_charset_map (charset, head, n_entries, control_flag);
+  for (entries = head; entries; entries = head)
+    {
+      head = entries->next; /* move the head of the list on */
+      xfree (entries);      /* free the old head            */
+    }
 }
 
 
-- 
1.6.6.1