Hi, Simon
Thanks for review.
> > + int has_efi = 0;
>
> The indentation of the line below is inconsistent with the line above.
Will fix
>
> > +
> > + has_efi = access("/sys/firmware/efi/systab", F_OK);
> > + if (has_efi < 0)
> > + return;
> > +
> > + esd = malloc(sizeof(struct efi_setup_data));
> > + if (!esd)
> > + return;
>
> This function appears to leak esd.
I left the error handling after testing but later forgot to add them.
Will fix
>
> > + memset(esd, 0, sizeof(struct efi_setup_data));
> > + get_efi_value(esd);
> > + nr_maps = get_efi_runtime_map(&esd);
> > + size = nr_maps * sizeof(struct efi_mem_descriptor) + sizeof(struct
> > efi_setup_data);
> > + sd = malloc(sizeof(struct setup_data) + size);
> > + if (!sd) {
> > + free(esd);
> > + return;
> > + }
>
> This function appears to leak sd.
Will fix
--
Thanks
Dave
_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec
