From 52d805117abd6c137a42c77b5678879cde631bb4 Mon Sep 17 00:00:00 2001 From: Julio Faracco Date: Fri, 18 Oct 2019 00:15:45 -0300 Subject: [PATCH] conf: Fix memory leak caused by missing VIR_FREE for video resolution. Commit 72862797 introduced resolution settings for QEMU video drivers. It includes a new structure inside video definition. So, the code needs to clear pointer allocation for that structure into clear function virDomainVideoDefClear(). This commit adds this missing VIR_FREE(). Reviewed-by: Jonathon Jongsma Signed-off-by: Julio Faracco --- src/conf/domain_conf.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 081124bee2..149a047446 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -2826,6 +2826,7 @@ virDomainVideoDefClear(virDomainVideoDefPtr def) if (def->accel) VIR_FREE(def->accel->rendernode); VIR_FREE(def->accel); + VIR_FREE(def->res); VIR_FREE(def->virtio); if (def->driver) VIR_FREE(def->driver->vhost_user_binary);