On 13.02.2015 21:32, Lunar wrote:
> Andrei Borzenkov:
>> Follow-up Comment #1, bug #44239 (project grub):
>>
>> Can't comment about behavior itself. Re. patch
>>
>> +    cmdline = grub_strdup(curentry->append);
>> Error check missing.
>>
>> +    grub_free(cmdline);
>> You need to delete free(cmdline) later.
> 
> Both fixed.
> 
>> Again, can you post updated version to grub-devel for easier review?
> 
> Here it is.
> 
The code at this place turned out to be not very understandable. I
simplified the code and did the change you requested.
> 
> 
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
> 


Attachment: signature.asc
Description: OpenPGP digital signature

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

Reply via email to