From ee9ad853574e3ee1a210db6207b84ecd466e08ee Mon Sep 17 00:00:00 2001 From: Keith Packard Date: Thu, 17 Sep 2009 17:57:40 -0700 Subject: [PATCH] Make sure DRM output properties are freed after we're done with them. drmmode_output_get_modes was fetching output properties but only freeing some of them. Signed-off-by: Keith Packard --- src/drmmode_display.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/drmmode_display.c b/src/drmmode_display.c index 04ba350b..cf88cd83 100644 --- a/src/drmmode_display.c +++ b/src/drmmode_display.c @@ -793,8 +793,12 @@ drmmode_output_get_modes(xf86OutputPtr output) /* look for an EDID property */ for (i = 0; i < koutput->count_props; i++) { props = drmModeGetProperty(drmmode->fd, koutput->props[i]); - if (!props || !(props->flags & DRM_MODE_PROP_BLOB)) + if (!props) continue; + if (!(props->flags & DRM_MODE_PROP_BLOB)) { + drmModeFreeProperty(props); + continue; + } if (!strcmp(props->name, "EDID")) { drmModeFreePropertyBlob(drmmode_output->edid_blob);