On Wed, Sep 04, 2013 at 11:58:10AM +0530, Manjunath Goudar wrote: > From: Jeyaraman R <[email protected]> > > 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 <[email protected]> > Signed-off-by: Manjunath Goudar <[email protected]> > Cc: David Airlie <[email protected]> > Cc: "Paul E. McKenney" <[email protected]> > Cc: David Howells <[email protected]> > Cc: Thomas Gleixner <[email protected]> > Cc: Dave Jones <[email protected]> > Cc: Dave Airlie <[email protected]> > Cc: [email protected] > Cc: [email protected]
Reviewed-by: Konrad Rzeszutek Wilk <[email protected]> > --- > 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 > > _______________________________________________ > dri-devel mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/dri-devel
