From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dongwon Kim Subject: [RFC PATCH 59/60] hyper_dmabuf: freeing hy_drv_priv when drv init fails (v2) Date: Tue, 19 Dec 2017 11:30:15 -0800 Message-ID: <1513711816-2618-59-git-send-email-dongwon.kim@intel.com> References: <1513711816-2618-1-git-send-email-dongwon.kim@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <1513711816-2618-1-git-send-email-dongwon.kim@intel.com> Sender: linux-kernel-owner@vger.kernel.org To: linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, xen-devel@lists.xenproject.org, mateuszx.potrola@intel.com, dongwon.kim@intel.com, Kai Chen List-Id: dri-devel@lists.freedesktop.org From: Kai Chen Make sure hy_drv_priv is freed before exiting in several places in hyper_dmabuf_drv_init. v2: unlocking mutex before freeing hy_drv_priv when bknd_ops->init fails Signed-off-by: Kai Chen Signed-off-by: Dongwon Kim --- drivers/xen/hyper_dmabuf/hyper_dmabuf_drv.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/xen/hyper_dmabuf/hyper_dmabuf_drv.c b/drivers/xen/hyper_dmabuf/hyper_dmabuf_drv.c index bbb3414..eead4c0 100644 --- a/drivers/xen/hyper_dmabuf/hyper_dmabuf_drv.c +++ b/drivers/xen/hyper_dmabuf/hyper_dmabuf_drv.c @@ -271,8 +271,10 @@ static int __init hyper_dmabuf_drv_init(void) return -ENOMEM; ret = register_device(); - if (ret < 0) + if (ret < 0) { + kfree(hy_drv_priv); return ret; + } /* currently only supports XEN hypervisor */ #ifdef CONFIG_HYPER_DMABUF_XEN @@ -284,6 +286,7 @@ static int __init hyper_dmabuf_drv_init(void) if (hy_drv_priv->bknd_ops == NULL) { printk(KERN_ERR "Hyper_dmabuf: no backend found\n"); + kfree(hy_drv_priv); return -1; } @@ -336,6 +339,8 @@ static int __init hyper_dmabuf_drv_init(void) if (ret < 0) { dev_dbg(hy_drv_priv->dev, "failed to initialize backend.\n"); + mutex_unlock(&hy_drv_priv->lock); + kfree(hy_drv_priv); return ret; } } -- 2.7.4