grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 2/2] second attempt to fix using grub device name as install


From: Andrey Borzenkov
Subject: Re: [PATCH 2/2] second attempt to fix using grub device name as install device
Date: Sat, 7 Dec 2013 13:40:22 +0400

В Sat, 07 Dec 2013 09:47:44 +0100
Vladimir 'φ-coder/phcoder' Serbinenko <address@hidden> пишет:

> On 07.12.2013 09:44, Andrey Borzenkov wrote:
> > +     install_device[grub_strlen (install_device) - 1] = '\0';
> > +     install_drive = xstrdup (install_device + 1);
> > +     install_device[grub_strlen (install_device) - 1] = ')';
> > +        }
> Bad code. You forgot that grub_strlen changes with your operations. You
> need to keep explicit pointer.
> 
> 

Oops. Any reason "our" gnulib is missing xstrndup? It is not the only
place where it will make things more simple and readable.

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]