2003-04-02 Havoc Pennington <hp@redhat.com>

	* dbus/dbus-string.c (set_length): fix a bug - we allocated max of
	current alloc and needed new length, not max of the doubled
	allocation and needed new length. Also, when building tests,
	don't do the double-allocation stuff, just realloc every time.

diff --git a/ChangeLog b/ChangeLog
index e72e8a6da12341d5e4540be6f3743fbfcadbbd3d..2340ba967b5c5dcaf3f635fc569115395d0ccde9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2003-04-02  Havoc Pennington  <hp@redhat.com>
+
+       * dbus/dbus-string.c (set_length): fix a bug - we allocated max of
+       current alloc and needed new length, not max of the doubled
+       allocation and needed new length. Also, when building tests, 
+       don't do the double-allocation stuff, just realloc every time.
+
 2003-04-02  Havoc Pennington  <hp@redhat.com>
 
        * dbus/dbus-sysdeps.c (_dbus_file_get_contents): include filenames

diff --git a/dbus/dbus-string.c b/dbus/dbus-string.c
index dd5781fa1d7b12949598a4633de447e899e6e32b..0bd754b39afa72229a42fe175a985c59b7365dda 100644 (file)
--- a/dbus/dbus-string.c
+++ b/dbus/dbus-string.c
@@ -361,8 +361,17 @@ set_length (DBusRealString *real,
       else
         new_allocated = real->allocated * 2;
 
+      /* if you change the code just above here, run the tests without
+       * the following before you commit
+       */
+#ifdef DBUS_BUILD_TESTS
+      new_allocated = 0; /* ensure a realloc every time so that we go
+                          * through all malloc failure codepaths
+                          */
+#endif
+      
       /* But be sure we always alloc at least space for the new length */
-      new_allocated = MAX (real->allocated, new_length + ALLOCATION_PADDING);
+      new_allocated = MAX (new_allocated, new_length + ALLOCATION_PADDING);
         
       new_str = dbus_realloc (real->str - real->align_offset, new_allocated);
       if (new_str == NULL)