[Top][All Lists]
[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
signature.asc
Description: OpenPGP digital signature
Re: Release of 2.00~rc1, David Cantrell, 2012/06/25
Re: Release of 2.00~rc1, Bruce Dubbs, 2012/06/26