(font_parse_xlfd): Check DPI and AVGWIDTH properties more rigidly.

diff --git a/src/ChangeLog b/src/ChangeLog
index 4dd53f21981de448c28427b03d9bd7b6df279227..523f19ff0c00d399c6fc68f3c336c002a71950af 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,11 @@
+2009-07-28  Kenichi Handa  <handa@m17n.org>
+
+       * font.c (font_parse_xlfd): Check DPI and AVGWIDTH properties more
+       rigidly.
+
+       * xfont.c (xfont_list_pattern): Don't ignore the return value of
+       font_parse_xlfd.  Check font properties more rigidly.
+
 2009-07-27  Dan Nicolaescu  <dann@ics.uci.edu>
 
        * s/netbsd.h (SIGNALS_VIA_CHARACTERS): Remove, already defined in bsd-common.h

diff --git a/src/font.c b/src/font.c
index d2cd1febe9355d0fb1011f45b849ea1262a2ae5d..cfc48523d12ba3cfdb83ef4c2ffd155ff234498f 100644 (file)
--- a/src/font.c
+++ b/src/font.c
@@ -1139,7 +1139,10 @@ font_parse_xlfd (name, font)
            }
        }
 
-      ASET (font, FONT_DPI_INDEX, INTERN_FIELD (XLFD_RESY_INDEX));
+      val = INTERN_FIELD (XLFD_RESY_INDEX);
+      if (! NILP (val) && ! INTEGERP (val))
+       return -1;
+      ASET (font, FONT_DPI_INDEX, val);
       val = INTERN_FIELD (XLFD_SPACING_INDEX);
       if (! NILP (val))
        {
@@ -1151,8 +1154,10 @@ font_parse_xlfd (name, font)
       p = f[XLFD_AVGWIDTH_INDEX];
       if (*p == '~')
        p++;
-      ASET (font, FONT_AVGWIDTH_INDEX,
-           font_intern_prop (p, f[XLFD_REGISTRY_INDEX] - 1 - p, 0));
+      val = font_intern_prop (p, f[XLFD_REGISTRY_INDEX] - 1 - p, 0);
+      if (! NILP (val) && ! INTEGERP (val))
+       return -1;
+      ASET (font, FONT_AVGWIDTH_INDEX, val);
     }
   else
     {