From: Matthias Clasen <mclasen@redhat.com>
Date: Wed, 3 May 2006 12:23:44 +0000 (+0000)
Subject: Use malloced memory instead of stack-allocated, so that MALLOC_CHECK_=2
X-Git-Url: http://git.openbox.org/?a=commitdiff_plain;h=92085171880640cf4bba0de6b78879b476a7b8cd;p=dana%2Fcg-glib.git

Use malloced memory instead of stack-allocated, so that MALLOC_CHECK_=2

2006-05-03  Matthias Clasen  <mclasen@redhat.com>

	* tests/base64-test.c (test_incremental): Use malloced memory
	instead of stack-allocated, so that MALLOC_CHECK_=2 catches
	the OOB write...
---

diff --git a/ChangeLog b/ChangeLog
index ef05f14b..914f60f5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2006-05-03  Matthias Clasen  <mclasen@redhat.com>
+
+	* tests/base64-test.c (test_incremental): Use malloced memory
+	instead of stack-allocated, so that MALLOC_CHECK_=2 catches
+	the OOB write...
+
+	* glib/gbookmarkfile.c (g_bookmark_file_load_from_data): Remove
+	an overzealous return_if_fail check that causes make check to 
+	fail.
+
 2006-05-02  Matthias Clasen  <mclasen@redhat.com>
 
 	* configure.in: Bump version
diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12
index ef05f14b..914f60f5 100644
--- a/ChangeLog.pre-2-12
+++ b/ChangeLog.pre-2-12
@@ -1,3 +1,13 @@
+2006-05-03  Matthias Clasen  <mclasen@redhat.com>
+
+	* tests/base64-test.c (test_incremental): Use malloced memory
+	instead of stack-allocated, so that MALLOC_CHECK_=2 catches
+	the OOB write...
+
+	* glib/gbookmarkfile.c (g_bookmark_file_load_from_data): Remove
+	an overzealous return_if_fail check that causes make check to 
+	fail.
+
 2006-05-02  Matthias Clasen  <mclasen@redhat.com>
 
 	* configure.in: Bump version
diff --git a/tests/base64-test.c b/tests/base64-test.c
index 6c458d2e..6484988e 100644
--- a/tests/base64-test.c
+++ b/tests/base64-test.c
@@ -11,13 +11,16 @@ static guchar data[DATA_SIZE];
 static void
 test_incremental (gboolean line_break)
 {
-  char text[DATA_SIZE * 2];
   char *p;
-  guchar data2[DATA_SIZE];
   int i;
   gsize len, decoded_len, max;
   int state, save;
   guint decoder_save;
+  char *text;
+  guchar *data2;
+
+  data2 = g_malloc (DATA_SIZE);
+  text = g_malloc (DATA_SIZE * 2);
 
   len = 0;
   state = 0;
@@ -65,6 +68,9 @@ test_incremental (gboolean line_break)
       g_print ("Wrong decoded base64 data\n");
       exit (1);
     }
+
+  g_free (text);
+  g_free (data2);
 }
 
 static void
@@ -90,6 +96,8 @@ test_full (void)
       g_print ("Wrong decoded base64 data\n");
       exit (1);
     }
+
+  g_free (data2);
 }
 
 int