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 have
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, a
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, bec
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 insertions(+), 2 deletions(-)
diff --git