]> code.delx.au - gnu-emacs/commitdiff
Fix memory leak in fontset handling
authorDima Kogan <dima@secretsauce.net>
Tue, 20 Oct 2015 15:12:55 +0000 (18:12 +0300)
committerEli Zaretskii <eliz@gnu.org>
Tue, 20 Oct 2015 15:12:55 +0000 (18:12 +0300)
* src/font.c (copy_font_spec): Make a deep copy of the input
argument FONT.  (Bug#21651)

src/font.c

index a52a653d29bec26c16269d16b2541f540faa068c..be55ce9bc7bf12f3dd528d1a6364efde995eef31 100644 (file)
@@ -3981,7 +3981,10 @@ copy_font_spec (Lisp_Object font)
   pcdr = spec->props + FONT_EXTRA_INDEX;
   for (tail = AREF (font, FONT_EXTRA_INDEX); CONSP (tail); tail = XCDR (tail))
     if (!EQ (XCAR (XCAR (tail)), QCfont_entity))
-      *pcdr = Fcons (XCAR (tail), Qnil), pcdr = xcdr_addr (*pcdr);
+      {
+        *pcdr = Fcons (Fcons (XCAR (XCAR (tail)), CDR (XCAR (tail))), Qnil);
+        pcdr = xcdr_addr (*pcdr);
+      }
 
   XSETFONT (new_spec, spec);
   return new_spec;