[Intel-gfx] [PATCH 3/9] drm/i915: fix memleak in intel_set_config_save_state()

2014-08-29 Thread Jani Nikula
On Fri, 29 Aug 2014, Chris Wilson wrote: > On Fri, Aug 29, 2014 at 10:38:43AM +0300, Jani Nikula wrote: >> On Thu, 28 Aug 2014, Gustavo Padovan wrote: >> > From: Gustavo Padovan >> > >> > If the save_encoder_crtcs or save_connector_encoders allocation fail >> > we need to free everything we

[PATCH 3/9] drm/i915: fix memleak in intel_set_config_save_state()

2014-08-29 Thread Jani Nikula
On Thu, 28 Aug 2014, Gustavo Padovan wrote: > From: Gustavo Padovan > > If the save_encoder_crtcs or save_connector_encoders allocation fail > we need to free everything we have already allocated. There is no memleak, because the caller of intel_set_config_save_state() checks the return value,

[Intel-gfx] [PATCH 3/9] drm/i915: fix memleak in intel_set_config_save_state()

2014-08-29 Thread Chris Wilson
On Fri, Aug 29, 2014 at 10:38:43AM +0300, Jani Nikula wrote: > On Thu, 28 Aug 2014, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > If the save_encoder_crtcs or save_connector_encoders allocation fail > > we need to free everything we have already allocated. > > There is no memleak,

[PATCH 3/9] drm/i915: fix memleak in intel_set_config_save_state()

2014-08-28 Thread Gustavo Padovan
From: Gustavo Padovan If the save_encoder_crtcs or save_connector_encoders allocation fail we need to free everything we have already allocated. Signed-off-by: Gustavo Padovan --- drivers/gpu/drm/i915/intel_display.c | 10 -- 1 file changed, 8