On Tue, 2025-08-05 at 08:07 +0200, Jan Kiszka wrote:
> 
> 
> I suspect this is the real fix:
> 
> diff --git a/env/env_config_file.c b/env/env_config_file.c
> index b0cf043..03c618a 100644
> --- a/env/env_config_file.c
> +++ b/env/env_config_file.c
> @@ -85,7 +85,7 @@ bool probe_config_file(CONFIG_PART *cfgpart)
>                 }
>                 if (do_unmount) {
>                         unmount_partition(cfgpart);
> -               } else {
> +               } else if (!result) {
>                         free(cfgpart->mountpoint);
>                         cfgpart->mountpoint = NULL;
>                 }
> 
> 
Yes, that fix is also working for me. Wasn't sure about the intention of that 
original change.
Another thing could be, to avoid printing "empty" values in case of an 
error...means handling the
return of read_env ... But I don't see this as very important..

> Jan
> 

-- 
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/0cbb5584a0f534a5e81bea1526f8ef334f38b8c4.camel%40linutronix.de.

Reply via email to