On 11 October 2016 at 10:04, Edward O'Callaghan
<[email protected]> wrote:
> Probably unlikely however ensure we don't leak a heap allocation
> on the fail path.
>
> Signed-off-by: Edward O'Callaghan <[email protected]>
> ---
>  src/amd/vulkan/winsys/amdgpu/radv_amdgpu_winsys.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_winsys.c 
> b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_winsys.c
> index 0ce44ac..ded5ed7 100644
> --- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_winsys.c
> +++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_winsys.c
> @@ -356,5 +356,6 @@ radv_amdgpu_winsys_create(int fd)
>
>         return &ws->base;
>  fail:
There's a missing amdgpu_device_deinitialize() in the error path(s) of
this function.

-Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to