Re: [patch] i915: fix small leak on error path

2010-03-17 Thread Eric Anholt
On Sat, 6 Mar 2010 12:50:54 +0100, Daniel Vetter wrote: > Oh, dang. Thanks for catching this. Eric, please merge. > > Cc: sta...@kernel.org (for .33) > Reviewed-by: Daniel Vetter > > On Sat, Mar 06, 2010 at 02:05:39PM +0300, Dan Carpenter wrote: > > We should free "params" before returning. > >

[patch] i915: fix small leak on error path

2010-03-06 Thread Dan Carpenter
We should free "params" before returning. Signed-off-by: Dan Carpenter diff --git a/drivers/gpu/drm/i915/intel_overlay.c b/drivers/gpu/drm/i915/intel_overlay.c index d355d1d..60595fc 100644 --- a/drivers/gpu/drm/i915/intel_overlay.c +++ b/drivers/gpu/drm/i915/intel_overlay.c @@ -1068,14 +1068,1

Re: [patch] i915: fix small leak on error path

2010-03-06 Thread Daniel Vetter
Oh, dang. Thanks for catching this. Eric, please merge. Cc: sta...@kernel.org (for .33) Reviewed-by: Daniel Vetter On Sat, Mar 06, 2010 at 02:05:39PM +0300, Dan Carpenter wrote: > We should free "params" before returning. > > Signed-off-by: Dan Carpenter > > diff --git a/drivers/gpu/drm/i915/