grub-devel
[Top][All Lists]
Advanced

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

Re: Release of 2.00~rc1


From: Vladimir 'φ-coder/phcoder' Serbinenko
Subject: Re: Release of 2.00~rc1
Date: Mon, 25 Jun 2012 20:23:11 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.5) Gecko/20120624 Icedove/10.0.5

On 24.06.2012 09:29, Robert Mabee wrote:

> On 06/23/2012 04:59 PM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
>> Hello, all. I'm proud to introduce 2.00~rc1 to you.
>> It's available as
>> ftp://alpha.gnu.org/gnu/grub/grub-2.00~rc1.tar.gz
>> or
>> ftp://alpha.gnu.org/gnu/grub/grub-2.00~rc1.tar.xz
>> As far as I'm aware all issues have been either postponed or resolved
>> correctly, so I'll release 2.00 on Monday. Give ~rc1 a spin over the
>> weekend and report any issues.
>>
> Missed this one, where the symbol belongs to some other interface,
> but has a value close enough to require an insane test (a filename
> containing a newline) to get incorrect results:
> 
> === modified file 'grub-core/commands/wildcard.c'
> --- old/grub-core/commands/wildcard.c 2012-06-08 20:54:21 +0000
> +++ new/grub-core/commands/wildcard.c 2012-06-24 06:55:33 +0000
> @@ -153,7 +153,7 @@
> buffer[i] = '\0';
> grub_dprintf ("expand", "Regexp is %s\n", buffer);
> 
> - if (regcomp (regexp, buffer, RE_SYNTAX_GNU_AWK))
> + if (regcomp (regexp, buffer, REG_EXTENDED))
> {
> grub_free (buffer);
> return 1;
> 

REG_EXTENDED isn't even a syntax type but is just refined to 1. Valid
syntaxes are:

RE_SYNTAX_EMACS
RE_SYNTAX_AWK
RE_SYNTAX_GNU_AWK                                               
RE_SYNTAX_POSIX_AWK                                             RE_SYNTAX_GREP
RE_SYNTAX_EGREP                                         RE_SYNTAX_POSIX_EGREP   
                                        RE_SYNTAX_ED
RE_SYNTAX_SED
RE_SYNTAX_POSIX_BASIC                                           
RE_SYNTAX_POSIX_MINIMAL_BASIC                           
RE_SYNTAX_POSIX_EXTENDED                                        
RE_SYNTAX_POSIX_MINIMAL_EXTENDED                                

Additionally no real testcase was ever demonstrated.

-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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