Re: [PATCH] RDMA/ucma: Fix resource leak on error path
From: Jason Gunthorpe
Date: Wed Sep 02 2020 - 20:32:52 EST
On Wed, Sep 02, 2020 at 07:34:26PM -0500, Gustavo A. R. Silva wrote:
> Hi Alex,
>
> On 9/2/20 11:24, Alex Dewar wrote:
> > In ucma_process_join(), if the call to xa_alloc() fails, the function
> > will return without freeing mc. Fix this by jumping to the correct line.
> >
> > In the process I renamed the jump labels to something more memorable for
> > extra clarity.
> >
> > Addresses-Coverity: ("Resource leak")
>
> If you are using a public Coverity scan, please also include the Coverity ID.
> In this case ID 1496814, something like:
>
> Addresses-Coverity-ID: 1496814 ("Resource leak")
Thanks, I fixed it up
Jason