From mboxrd@z Thu Jan 1 00:00:00 1970 From: Manjunath Goudar Subject: [PATCH] DRM: TTM: Fix memory leak issue in ttm_agp_tt_create(). Date: Wed, 4 Sep 2013 11:58:10 +0530 Message-ID: <1378276090-10130-1-git-send-email-manjunath.goudar@linaro.org> Return-path: Sender: linux-kernel-owner@vger.kernel.org To: patches@linaro.org Cc: linaro-kernel@lists.linaro.org, dsaxena@linaro.org, Jeyaraman R , Manjunath Goudar , David Airlie , "Paul E. McKenney" , David Howells , Thomas Gleixner , Dave Jones , Dave Airlie , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org From: Jeyaraman R This patch adds kfree() in ttm_agp_tt_create() to avoide the memory leak, without this there is a chance of memory leak in ttm_tt_init() fail case. Signed-off-by: Jeyaraman R Signed-off-by: Manjunath Goudar Cc: David Airlie Cc: "Paul E. McKenney" Cc: David Howells Cc: Thomas Gleixner Cc: Dave Jones Cc: Dave Airlie Cc: dri-devel@lists.freedesktop.org Cc: linux-kernel@vger.kernel.org --- drivers/gpu/drm/ttm/ttm_agp_backend.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/ttm/ttm_agp_backend.c b/drivers/gpu/drm/ttm/ttm_agp_backend.c index 3302f99..764be36 100644 --- a/drivers/gpu/drm/ttm/ttm_agp_backend.c +++ b/drivers/gpu/drm/ttm/ttm_agp_backend.c @@ -126,6 +126,7 @@ struct ttm_tt *ttm_agp_tt_create(struct ttm_bo_device *bdev, agp_be->ttm.func = &ttm_agp_func; if (ttm_tt_init(&agp_be->ttm, bdev, size, page_flags, dummy_read_page)) { + kfree(agp_be); return NULL; } -- 1.8.1.2