On 31.07.25 14:07, INgo Rah wrote:
> Without this fix configurations cannot be read or written on
> already mounted partitions because the filename is missing.
> Freeing the memory is anyhow done in bgenv_finalize at the very end.
>
> Signed-off-by: INgo Rah <ingo....@linutronix.de>
>
> diff --git a/env/env_config_file.c b/env/env_config_file.c
> index b0cf043..e859575 100644
> --- a/env/env_config_file.c
> +++ b/env/env_config_file.c
> @@ -85,9 +85,6 @@ bool probe_config_file(CONFIG_PART *cfgpart)
> }
> if (do_unmount) {
> unmount_partition(cfgpart);
> - } else {
> - free(cfgpart->mountpoint);
> - cfgpart->mountpoint = NULL;
> }
> return result;
> }
>
Can you please explain the code path where this is causing a regression?
Are you calling probe_config_file() directly, or is it something
happening within plain libebgenv as well?
Jan
--
Siemens AG, Foundational Technologies
Linux Expert Center
--
You received this message because you are subscribed to the Google Groups "EFI
Boot Guard" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to efibootguard-dev+unsubscr...@googlegroups.com.
To view this discussion visit
https://groups.google.com/d/msgid/efibootguard-dev/33452c0a-997e-4f23-a408-750e2eb7e001%40siemens.com.
