lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 31 Jul 2009 16:55:56 +0800 From: Xiaotian Feng <dfeng@...hat.com> To: airlied@...ux.ie, dri-devel@...ts.sourceforge.net Cc: linux-kernel@...r.kernel.org, Xiaotian Feng <dfeng@...hat.com> Subject: [PATCH 2/4] gpu/drm: fix memory leak in drm_fill_in_dev fail path In drm_fill_in_dev, drm_ht_create/drm_agp_init/drm_gem_init all allocate memories. but in the fail path, those memory is not freed, then memory leaks is resulted. Signed-off-by: Xiaotian Feng <dfeng@...hat.com> --- drivers/gpu/drm/drm_stub.c | 25 ++++++++++++++++++++----- 1 files changed, 20 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c index 55bb8a8..8ac1ddb 100644 --- a/drivers/gpu/drm/drm_stub.c +++ b/drivers/gpu/drm/drm_stub.c @@ -259,7 +259,7 @@ static int drm_fill_in_dev(struct drm_device * dev, struct pci_dev *pdev, && (dev->agp == NULL)) { DRM_ERROR("Cannot initialize the agpgart module.\n"); retcode = -EINVAL; - goto error_out_unreg; + goto error_agp; } if (drm_core_has_MTRR(dev)) { if (dev->agp) @@ -274,7 +274,7 @@ static int drm_fill_in_dev(struct drm_device * dev, struct pci_dev *pdev, retcode = drm_ctxbitmap_init(dev); if (retcode) { DRM_ERROR("Cannot allocate memory for context bitmap.\n"); - goto error_out_unreg; + goto error_ctxbitmap; } if (driver->driver_features & DRIVER_GEM) { @@ -282,14 +282,29 @@ static int drm_fill_in_dev(struct drm_device * dev, struct pci_dev *pdev, if (retcode) { DRM_ERROR("Cannot initialize graphics execution " "manager (GEM)\n"); - goto error_out_unreg; + goto error_gem; } } return 0; - error_out_unreg: - drm_lastclose(dev); +error_gem: + drm_ctxbitmap_cleanup(dev); +error_ctxbitmap: + if (drm_core_has_MTRR(dev) && drm_core_has_AGP(dev) && + dev->agp && dev->agp->agp_mtrr >= 0) { + int retval; + retval = mtrr_del(dev->agp->agp_mtrr, + dev->agp->agp_info.aper_base, + dev->agp->agp_info.aper_size * 1024 * 1024); + DRM_DEBUG("mtrr_del=%d\n", retval); + } + if (drm_core_has_AGP(dev) && dev->agp) { + kfree(dev->agp); + dev->agp = NULL; + } +error_agp: + drm_ht_remove(&dev->map_hash); return retcode; } -- 1.6.2.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists