[dm-devel] [PATCH 03/15] Fix leak in mpathpersist
Martin Wilck
mwilck at suse.com
Fri Jan 17 15:53:47 UTC 2020
On Thu, 2020-01-16 at 20:18 -0600, Benjamin Marzinski wrote:
> If the persistent in command fails, the response buffer must be
> freed.
> Found by Coverity
>
> Signed-off-by: Benjamin Marzinski <bmarzins at redhat.com>
> ---
> mpathpersist/main.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/mpathpersist/main.c b/mpathpersist/main.c
> index 278b8d51..920e686c 100644
> --- a/mpathpersist/main.c
> +++ b/mpathpersist/main.c
> @@ -499,6 +499,7 @@ static int handle_args(int argc, char * argv[],
> int nline)
> if (ret != MPATH_PR_SUCCESS )
> {
> fprintf (stderr, "Persistent Reserve IN command
> failed\n");
> + free(resp);
> goto out_fd;
> }
>
Reviewed-by: Martin Wilck <mwilck at suse.com>
More information about the dm-devel
mailing list