On Tue, Jun 20, 2023 at 04:03:52PM +0200, Phil Sutter wrote:
> Missing call to nft_ctx_free() upsets valgrind enough to suspect
> possible losses, add them where sensible. This fixes reports with
> readline-lined builds at least. The same code is shared for libedit
> though, and there's an obvious spot for linenoise.
Maybe call nft_ctx_free() from cli_exit() ?
> Signed-off-by: Phil Sutter <phil@xxxxxx>
> ---
> src/cli.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/cli.c b/src/cli.c
> index 11fc85abeaa2b..bc7f64ef0b762 100644
> --- a/src/cli.c
> +++ b/src/cli.c
> @@ -126,6 +126,7 @@ static void cli_complete(char *line)
> if (line == NULL) {
> printf("\n");
> cli_exit();
> + nft_ctx_free(cli_nft);
> exit(0);
> }
>
> @@ -141,6 +142,7 @@ static void cli_complete(char *line)
>
> if (!strcmp(line, CMDLINE_QUIT)) {
> cli_exit();
> + nft_ctx_free(cli_nft);
> exit(0);
> }
>
> @@ -244,6 +246,7 @@ int cli_init(struct nft_ctx *nft)
> linenoiseFree(line);
> }
> cli_exit();
> + nft_ctx_free(nft);
> exit(0);
> }
>
> --
> 2.40.0
>
