<div dir="ltr">Looks good.<br><br>Reviewed-by: Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net">jstpierre@mecheye.net</a>><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Apr 7, 2014 at 1:47 PM, Adam Jackson <span dir="ltr"><<a href="mailto:ajax@redhat.com" target="_blank">ajax@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Adam Jackson <<a href="mailto:ajax@redhat.com">ajax@redhat.com</a>><br>
---<br>
hw/dmx/dmxfont.c | 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/hw/dmx/dmxfont.c b/hw/dmx/dmxfont.c<br>
index 6b81826..115422d 100644<br>
--- a/hw/dmx/dmxfont.c<br>
+++ b/hw/dmx/dmxfont.c<br>
@@ -397,12 +397,12 @@ dmxBELoadFont(ScreenPtr pScreen, FontPtr pFont)<br>
"font-path-related options, see the Xdmx man page.\n");<br>
}<br>
<br>
+ free(goodfps);<br>
if (!dmxIgnoreBadFontPaths ||<br>
(dmxIgnoreBadFontPaths && dmxSetFontPath(dmxScreen))) {<br>
/* We still have errors so return with error */<br>
dmxFreeFontPath(fp);<br>
XFreeFontPath(oldFontPath);<br>
- free(goodfps);<br>
return FALSE;<br>
}<br>
}<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.9.0<br>
<br>
_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br> Jasper<br>
</div>