Looks good.

Reviewed-by: Jasper St. Pierre <[email protected]>


On Mon, Apr 7, 2014 at 1:47 PM, Adam Jackson <[email protected]> wrote:

> Signed-off-by: Adam Jackson <[email protected]>
> ---
>  hw/dmx/dmxfont.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/hw/dmx/dmxfont.c b/hw/dmx/dmxfont.c
> index 6b81826..115422d 100644
> --- a/hw/dmx/dmxfont.c
> +++ b/hw/dmx/dmxfont.c
> @@ -397,12 +397,12 @@ dmxBELoadFont(ScreenPtr pScreen, FontPtr pFont)
>                     "font-path-related options, see the Xdmx man page.\n");
>          }
>
> +        free(goodfps);
>          if (!dmxIgnoreBadFontPaths ||
>              (dmxIgnoreBadFontPaths && dmxSetFontPath(dmxScreen))) {
>              /* We still have errors so return with error */
>              dmxFreeFontPath(fp);
>              XFreeFontPath(oldFontPath);
> -            free(goodfps);
>              return FALSE;
>          }
>      }
> --
> 1.9.0
>
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>



-- 
  Jasper
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to