From: Simon Glass <sjg@chromium.org>
Date: Mon, 22 Feb 2016 04:10:26 +0000 (-0700)
Subject: video: freetype: Fix a memory leak with a bad parameter
X-Git-Tag: v2016.03-rc3~58^2~1
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=d5b9d11cb61c9b50cc799fc00b4e7c3a26004538;p=u-boot

video: freetype: Fix a memory leak with a bad parameter

Make sure to free memory used when the scale facture is incorrect.

Reported-by: Coverity (CID: 24068)
Signed-off-by: Simon Glass <sjg@chromium.org>
---

diff --git a/drivers/video/stb_truetype.h b/drivers/video/stb_truetype.h
index 91d8e6f905..26e483cf56 100644
--- a/drivers/video/stb_truetype.h
+++ b/drivers/video/stb_truetype.h
@@ -2426,7 +2426,10 @@ STBTT_DEF unsigned char *stbtt_GetGlyphBitmapSubpixel(const stbtt_fontinfo *info
 
    if (scale_x == 0) scale_x = scale_y;
    if (scale_y == 0) {
-      if (scale_x == 0) return NULL;
+      if (scale_x == 0) {
+         STBTT_free(vertices, info->userdata);
+         return NULL;
+      }
       scale_y = scale_x;
    }