Messages in this thread | | | From | "Koenig, Christian" <> | Subject | Re: [PATCH] drm/amdgpu: fix memory leak | Date | Fri, 4 Oct 2019 11:30:38 +0000 |
| |
Am 04.10.19 um 13:26 schrieb Das, Nirmoy: > On 10/4/19 1:13 PM, Koenig, Christian wrote: >>>> NAK, that is a double free. The bo list entries are freed by >>>> amdgpu_bo_list_put(). >>> Thanks, didn't realize that. >> Wait a second, what entries are you talking about? >> >> The entries in the list object are freed when amdgpu_bo_list_put() is >> called, but the temporary info array with the handles needs to be freed >> as well. >> >> And it looks like that is indeed leaked here. > I am talking about the `info` array created by > amdgpu_bo_create_list_entry_array().
Yeah, that are the handles and not the entries. Sorry that I was confused about that.
Your patch is correct, you should just update the commit message a bit.
BTW: Could you cleanup error handling here a bit more?
E.g. add an error_put_list handle and drop the "if (info)" and instead return directly if we fail to allocate info.
Thanks, Christian.
>> Regards, >> Christian. >> >>>> Regards, >>>> Christian. >>> Regards, >>> >>> Nirmoy >>> >>>>> } >>>>> >>>>> handle = r;
| |