Re: [PATCH] drm/amd/display - Don't leak memory when updating streams

2019-01-28 Thread Wentland, Harry
On 2019-01-28 9:00 a.m., Nicholas Kazlauskas wrote: > [Why] > The flip and full structures were allocated but never freed. > > [How] > Free them at the end of the function. There's a small behavioral > change here with the function returning early if the allocation fails > but we wouldn't should

Re: [PATCH] drm/amd/display - Don't leak memory when updating streams

2019-01-28 Thread Michel Dänzer
On 2019-01-28 3:00 p.m., Nicholas Kazlauskas wrote: > [Why] > The flip and full structures were allocated but never freed. > > [How] > Free them at the end of the function. There's a small behavioral > change here with the function returning early if the allocation fails > but we wouldn't should

Re: [PATCH] drm/amd/display - Don't leak memory when updating streams

2019-01-28 Thread Li, Sun peng (Leo)
On 2019-01-28 9:00 a.m., Nicholas Kazlauskas wrote: > [Why] > The flip and full structures were allocated but never freed. > > [How] > Free them at the end of the function. There's a small behavioral > change here with the function returning early if the allocation fails > but we wouldn't

[PATCH] drm/amd/display - Don't leak memory when updating streams

2019-01-28 Thread Nicholas Kazlauskas
[Why] The flip and full structures were allocated but never freed. [How] Free them at the end of the function. There's a small behavioral change here with the function returning early if the allocation fails but we wouldn't should be doing anything in that case anyway. Fixes: c00e0cc0fdc0