Hello Daniel,

Thanks a lot for pointing out these issues. I'll fix them in v2.

>> +
>> +      linkbuf = xmalloc (size+1);
> 
> Please check for NULL here. And "size + 1". In general I will not accept
> any patch which does not check for NULL for malloc() et consortes.
> 

Yes, I'll be more careful when reviewing our downstream patches before posting.

Best regards,
-- 
Javier Martinez Canillas
Software Engineer - Desktop Hardware Enablement
Red Hat

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

Reply via email to