summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAdam Jackson <ajax@redhat.com>2018-01-31 10:48:08 -0500
committerAdam Jackson <ajax@redhat.com>2018-02-15 14:27:24 -0500
commit2cb984533fd7a36151bb8be18a63f678a552ad71 (patch)
tree0cc6df6a4a171b1c97ff13e85c97a7a67b823416
parent9c4f6c7ab011435a492dd5cb1847bf00f290c219 (diff)
downloadxorg-driver-xf86-video-vesa-2cb984533fd7a36151bb8be18a63f678a552ad71.tar.gz
Use VBEFreeVBEInfo not free
A VbeInfoBlock has substructure, just freeing the object will leak. Unfortunately VBEFreeVBEInfo does not check for NULL first so we have to. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=25029 Signed-off-by: Adam Jackson <ajax@redhat.com>
-rw-r--r--src/vesa.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/vesa.c b/src/vesa.c
index 3f5b81c..2d3c10d 100644
--- a/src/vesa.c
+++ b/src/vesa.c
@@ -596,7 +596,8 @@ VESAFreeRec(ScrnInfoPtr pScrn)
}
#endif
free(pVesa->monitor);
- free(pVesa->vbeInfo);
+ if (pVesa->vbeInfo)
+ VBEFreeVBEInfo(pVesa->vbeInfo);
free(pVesa->pal);
free(pVesa->savedPal);
free(pVesa->fonts);