Fixed bugs in ns_get_family and ns_descriptor_to_entity that were caused by using...

diff --git a/src/ChangeLog b/src/ChangeLog
index a088c1b42d0d39b7993152ae6a173d1c16b20a13..764b345bd25a865ae94569b94ebbc1eccb816bda 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,13 @@
+2011-04-16  Ben Key  <bkey76@gmail.com>
+
+       * nsfont.m: Fixed bugs in ns_get_family and
+       ns_descriptor_to_entity that were caused by using free to
+       deallocate memory blocks that were allocated by xmalloc (via
+       xstrdup).  This caused Emacs to crash when compiled with
+       XMALLOC_OVERRUN_CHECK defined (when Emacs was configured with
+       --enable-checking=xmallocoverrun).  xfree is now used to
+       deallocate these memory blocks.
+
 2011-04-15  Paul Eggert  <eggert@cs.ucla.edu>
 
        * sysdep.c (emacs_read): Remove unnecessary check vs MAX_RW_COUNT.

diff --git a/src/nsfont.m b/src/nsfont.m
index 48d40223553a57d5a9f84106d093387feb683f4d..76c70aadf9ff863d56f71b52c9e8bf8f2eb79cdb 100644 (file)
--- a/src/nsfont.m
+++ b/src/nsfont.m
@@ -104,7 +104,7 @@ ns_get_family (Lisp_Object font_spec)
       NSString *family;
       ns_unescape_name (tmp);
       family = [NSString stringWithUTF8String: tmp];
-      free (tmp);
+      xfree (tmp);
       return family;
     }
 }
@@ -217,7 +217,7 @@ ns_descriptor_to_entity (NSFontDescriptor *desc,
        debug_print (font_entity);
       }
 
-    free (escapedFamily);
+    xfree (escapedFamily);
     return font_entity;
 }