On Mon, Aug 11, 2025 at 11:04:03PM +0530, Sudhakar Kuppusamy wrote:
> Thank you Daniel.
>
> > On 11 Aug 2025, at 10:51 PM, Daniel Kiper <dki...@net-space.pl> wrote:
> > On Tue, Jul 29, 2025 at 08:21:48PM +0530, Sudhakar Kuppusamy wrote:

[...]

> >> +  else if (grub_pks_use_keystore == true && check_sigs == true)
> >> +    {
> >> +      rc = create_db_list ();
> >> +      if (rc != GRUB_ERR_NONE)
> >> +        {
> >> +          free_db_list ();
> >
> > I would not call free_db_list() here because even partially populated
> > list can be useful. Though we should be sure partial or even empty list
> > does not lead to crashes.
>
> Sure. Do I need to use partial or empty list even out of memory case?

I would not try to differentiate failures here. Releasing a few kB of
memory will not help much in case of out of memory errors.

Daniel

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to