]> git.ipfire.org Git - thirdparty/grub.git/commitdiff
* util/grub-gen-asciih.c (add_glyph): Fix uninitialised variable.
authorTrevor Woerner <trevor.woerner@linaro.org>
Sat, 21 Jun 2014 18:26:47 +0000 (20:26 +0200)
committerVladimir Serbinenko <phcoder@gmail.com>
Sat, 21 Jun 2014 18:26:47 +0000 (20:26 +0200)
ChangeLog
util/grub-gen-asciih.c

index 6d2f8c635ca1a1d668f628f378d64eb63db1e12d..8929dbbbbb78459f96377b228e03ac721a7d4c3a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-21  Trevor Woerner <trevor.woerner@linaro.org>
+
+       * util/grub-gen-asciih.c (add_glyph): Fix uninitialised variable.
+
 2014-06-21  Vladimir Serbinenko  <phcoder@gmail.com>
 
        * grub-core/commands/verify.c (grub_pubkey_open): Trust procfs.
index e01447ae94fb9653891649c95593935192bae66a..e35dcb78fe610191598652d0feca8b998bc41198 100644 (file)
@@ -81,14 +81,14 @@ add_glyph (FT_UInt glyph_idx, FT_Face face,
   err = FT_Load_Glyph (face, glyph_idx, flag);
   if (err)
     {
-      printf ("Freetype Error %d loading glyph 0x%x for U+0x%x",
+      fprintf (stderr, "Freetype Error %d loading glyph 0x%x for U+0x%x",
              err, glyph_idx, char_code);
 
       if (err > 0 && err < (signed) ARRAY_SIZE (ft_errmsgs))
-       printf (": %s\n", ft_errmsgs[err]);
+       fprintf (stderr, ": %s\n", ft_errmsgs[err]);
       else
-       printf ("\n");
-      return;
+       fprintf (stderr, "\n");
+      exit (1);
     }
 
   glyph = face->glyph;
@@ -128,7 +128,6 @@ add_glyph (FT_UInt glyph_idx, FT_Face face,
 static void
 write_font_ascii_bitmap (FILE *file, FT_Face face)
 {
-  struct grub_glyph_info glyph;
   int char_code;
 
   fprintf (file, "/* THIS CHUNK OF BYTES IS AUTOMATICALLY GENERATED */\n");
@@ -138,10 +137,14 @@ write_font_ascii_bitmap (FILE *file, FT_Face face)
   for (char_code = 0; char_code <= 0x7f; char_code++)
     {
       FT_UInt glyph_idx;
-      
+      struct grub_glyph_info glyph;
+
       glyph_idx = FT_Get_Char_Index (face, char_code);
       if (!glyph_idx)
        return;
+
+      memset (&glyph, 0, sizeof(glyph));
+
       add_glyph (glyph_idx, face, char_code, &glyph);
 
       if (glyph.width == 8 && glyph.height == 16