test/cmdline: fix memory leak
Checks
Commit Message
Previous change wrongly removed a valid cmdline_free().
Fixes: 6ad06203a587 ("cmdline: free on exit")
Signed-off-by: David Marchand <david.marchand@redhat.com>
---
app/test/test_cmdline_lib.c | 2 ++
1 file changed, 2 insertions(+)
Comments
On Fri, Oct 29, 2021 at 01:23:15PM +0200, David Marchand wrote:
> Previous change wrongly removed a valid cmdline_free().
>
> Fixes: 6ad06203a587 ("cmdline: free on exit")
>
> Signed-off-by: David Marchand <david.marchand@redhat.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
Thanks!
On Fri, Oct 29, 2021 at 2:13 PM Olivier Matz <olivier.matz@6wind.com> wrote:
>
> On Fri, Oct 29, 2021 at 01:23:15PM +0200, David Marchand wrote:
> > Previous change wrongly removed a valid cmdline_free().
> >
> > Fixes: 6ad06203a587 ("cmdline: free on exit")
> >
> > Signed-off-by: David Marchand <david.marchand@redhat.com>
> Acked-by: Olivier Matz <olivier.matz@6wind.com>
Applied, thanks.
@@ -176,6 +176,8 @@ test_cmdline_socket_fns(void)
printf("Error: failed to open /dev/null for reading!");
return -1;
}
+ cmdline_free(cl);
+ cl = NULL;
/* void functions */
cmdline_stdin_exit(NULL);