randrproto libXau libxcb mesa mkfontscale pixman git update 5 Jan 2014

xserver          commit 2ea973e12f5d954211e1d10085a4c74581b43aca
libxcb           commit 3b72a2c9d1d656c74c691a45689e1d637f669e3a
libxcb/xcb-proto commit 0d8f09b6469c45be08dae3665d352269ffcc7df2
xkeyboard-config commit 8f49a59f8cd118564ec1554d428177d681bc8b7f
libX11           commit 3d69b0a83e62f8f6fbdd952fc49cdbdf8825e1e6
libXdmcp         commit 089081dca4ba3598c6f9bf401c029378943b5854
libXext          commit bb24f2970f2e425f4df90c9b73d078ad15a73fbb
libfontenc       commit 3acba630d8b57084f7e92c15732408711ed5137a
libXinerama      commit edd95182b26eb5d576d4878c559e0f17dddaa909
libXau           commit 1e4635be11154dd8262f37b379511bd627defa2a
xkbcomp          commit e3e6e938535532bfad175c1635256ab7fb3ac943
pixman           commit 82d094654a46bd97d47f1f132a01ae0a74b986f3
xextproto        commit 66afec3f49e8eb0d4c2e9af7088fc3116d4bafd7
randrproto       commit 19fc4c5a72eb9919d720ad66734029d9f8e313b1
glproto          commit f84853d97d5749308992412a215fa518b6536eb3
mkfontscale      commit eac564e0fc9052a39981ea47b271f7f3d2821944
xwininfo         commit ba0d1b0da21d2dbdd81098ed5778f3792b472e13
libXft           commit 4acfdaf95adb0a05c2a25550bdde036c865902f4
libXmu           commit 22d9c590901e121936f50dee97dc60c4f7defb63
libxtrans        commit 2c0a7840a28ae696e80e73157856d7a049fdf6c7
fontconfig       commit 5c725f2f5829238d16116f782d00d8bb0defaf08
mesa             commit a61ae2aa01c9b1976ba4107c1564e0c3f94aea4d

2 files changed, 22 insertions, 9 deletions

diff --git a/mkfontscale/list.c b/mkfontscale/list.c
index 1614d9537..320ce20c6 100644
--- a/mkfontscale/list.c
+++ b/mkfontscale/list.c
@@ -235,7 +235,12 @@ sortList(ListPtr old)
     int i;
     int l = listLength(old);
     ListPtr n;
-    ListPtr *sorted = malloc(l * sizeof(ListPtr));
+    ListPtr *sorted;
+
+    if (l <= 0)
+        return old;
+
+    sorted = malloc(l * sizeof(ListPtr));
 
     if (sorted == NULL)
         return old;
diff --git a/mkfontscale/mkfontscale.c b/mkfontscale/mkfontscale.c
index a494f931d..265f91369 100644
--- a/mkfontscale/mkfontscale.c
+++ b/mkfontscale/mkfontscale.c
@@ -490,7 +490,7 @@ safe(const char* s)
         i++;
     }
 
-    if(safe_flag) return s;
+    if(safe_flag) return strdup(s);
 
     len = i;
     t = malloc(len + 1);
@@ -514,7 +514,7 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
 {
     ListPtr xlfd = NULL;
     const char *foundry, *family, *weight, *slant, *sWidth, *adstyle,
-        *spacing, *full_name;
+        *spacing, *full_name, *tmp;
     TT_Header *head;
     TT_HoriHeader *hhea;
     TT_OS2 *os2;
@@ -582,11 +582,11 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
 
     if(t1info) {
         if(!family)
-            family = t1info->family_name;
+            family = strdup(t1info->family_name);
         if(!family)
-            family = t1info->full_name;
+            family = strdup(t1info->full_name);
         if(!full_name)
-            full_name = t1info->full_name;
+            full_name = strdup(t1info->full_name);
         if(!foundry)
             foundry = notice_foundry(t1info->notice);
         if(!weight)
@@ -603,7 +603,7 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
 
     if(!full_name) {
         fprintf(stderr, "Couldn't determine full name for %s\n", filename);
-        full_name = filename;
+        full_name = strdup(filename);
     }
 
     if(head) {
@@ -628,11 +628,13 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
         notice = getName(face, TT_NAME_ID_TRADEMARK);
         if(notice) {
             foundry = notice_foundry(notice);
+            free(notice);
         }
         if(!foundry) {
             notice = getName(face, TT_NAME_ID_MANUFACTURER);
             if(notice) {
                 foundry = notice_foundry(notice);
+                free(notice);
             }
         }
     }
@@ -650,7 +652,7 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
     if(!foundry) foundry = "misc";
     if(!family) {
         fprintf(stderr, "Couldn't get family name for %s\n", filename);
-        family = filename;
+        family = strdup(filename);
     }
 
     if(!weight) weight = "medium";
@@ -659,9 +661,11 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
     if(!adstyle) adstyle = "";
     if(!spacing) spacing = "p";
 
-    /* Yes, it's a memory leak. */
     foundry = safe(foundry);
+
+    tmp = family;
     family = safe(family);
+    free((void *)tmp);
 
     if(!isBitmap) {
         xlfd = listConsF(xlfd,
@@ -684,6 +688,10 @@ makeXLFD(char *filename, FT_Face face, int isBitmap)
                              spacing, 60);
         }
     }
+
+    free((void *)family);
+    free((void *)foundry);
+    free((void *)full_name);
     return xlfd;
 }