Rebuild with newer pesign so we'll get signed with the final signing keys.

Signed-off-by: Peter Jones <pjones@redhat.com>
This commit is contained in:
Peter Jones 2012-10-22 14:06:22 -04:00
parent f2e5630d7d
commit 1a0dbd0641
2 changed files with 88 additions and 48 deletions

View file

@ -1,19 +1,21 @@
From d08e2511db353b2db9c5785d3f22079674abd708 Mon Sep 17 00:00:00 2001 From 20ce60f211cef5f2c553130ba24b049381915252 Mon Sep 17 00:00:00 2001
From: Peter Jones <pjones@redhat.com> From: Peter Jones <pjones@redhat.com>
Date: Mon, 1 Oct 2012 13:24:37 -0400 Date: Mon, 1 Oct 2012 13:24:37 -0400
Subject: [PATCH] Pass "\x[[:hex:]][[:hex:]]" straight through unmolested. Subject: [PATCH] Handle "\x[[:hex:]][[:hex:]]" style escapes as well.
Sometimes we need to escape spaces and such in the config file, and we
do so by passing them in hex.
--- ---
grub-core/commands/wildcard.c | 16 +++++++++++++++- grub-core/commands/wildcard.c | 26 +++++++++++++++++++-
grub-core/lib/cmdline.c | 34 +++++++++++++++++++++++++++++++-- grub-core/lib/cmdline.c | 44 ++++++++++++++++++++++++++++++++--
grub-core/script/execute.c | 44 +++++++++++++++++++++++++++++++++++++------ grub-core/script/execute.c | 55 ++++++++++++++++++++++++++++++++++++++-----
3 files changed, 85 insertions(+), 9 deletions(-) 3 files changed, 116 insertions(+), 9 deletions(-)
diff --git a/grub-core/commands/wildcard.c b/grub-core/commands/wildcard.c diff --git a/grub-core/commands/wildcard.c b/grub-core/commands/wildcard.c
index 2b73d9a..d1235dc 100644 index 2b73d9a..8ea76c9 100644
--- a/grub-core/commands/wildcard.c --- a/grub-core/commands/wildcard.c
+++ b/grub-core/commands/wildcard.c +++ b/grub-core/commands/wildcard.c
@@ -420,6 +420,12 @@ check_file (const char *dir, const char *basename) @@ -420,6 +420,23 @@ check_file (const char *dir, const char *basename)
return found; return found;
} }
@ -22,21 +24,31 @@ index 2b73d9a..d1235dc 100644
+{ +{
+ return ((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F')); + return ((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F'));
+} +}
+
+static grub_int32_t
+hex_to_int(char c)
+{
+ int i = c | 0x20;
+ if (i >= '0' && i < '9')
+ return i - '0';
+ if (i >= 'a' && i < 'f')
+ return i - ('a' - 10);
+ return -1;
+}
+ +
static void static void
unescape (char *out, const char *in, const char *end) unescape (char *out, const char *in, const char *end)
{ {
@@ -428,7 +434,15 @@ unescape (char *out, const char *in, const char *end) @@ -428,7 +445,14 @@ unescape (char *out, const char *in, const char *end)
for (optr = out, iptr = in; iptr < end;) for (optr = out, iptr = in; iptr < end;)
{ {
- if (*iptr == '\\' && iptr + 1 < end) - if (*iptr == '\\' && iptr + 1 < end)
+ if (*iptr == '\\' && iptr + 3 < end && iptr[1] == 'x' && is_hex(iptr[2]) && is_hex(iptr[3])) + if (*iptr == '\\' && iptr + 3 < end && iptr[1] == 'x' && is_hex(iptr[2]) && is_hex(iptr[3]))
+ { + {
+ *optr++ = *iptr++; + int a = (hex_to_int(iptr[2]) << 4) | (hex_to_int(iptr[3]) & 0xf);
+ *optr++ = *iptr++; + *optr++ = a & 0xff;
+ *optr++ = *iptr++; + iptr += 4;
+ *optr++ = *iptr++;
+ continue; + continue;
+ } + }
+ else if (*iptr == '\\' && iptr + 1 < end) + else if (*iptr == '\\' && iptr + 1 < end)
@ -44,7 +56,7 @@ index 2b73d9a..d1235dc 100644
*optr++ = iptr[1]; *optr++ = iptr[1];
iptr += 2; iptr += 2;
diff --git a/grub-core/lib/cmdline.c b/grub-core/lib/cmdline.c diff --git a/grub-core/lib/cmdline.c b/grub-core/lib/cmdline.c
index a702e64..c8605a7 100644 index a702e64..78a96fa 100644
--- a/grub-core/lib/cmdline.c --- a/grub-core/lib/cmdline.c
+++ b/grub-core/lib/cmdline.c +++ b/grub-core/lib/cmdline.c
@@ -20,6 +20,12 @@ @@ -20,6 +20,12 @@
@ -75,27 +87,44 @@ index a702e64..c8605a7 100644
size++; size++;
else if (*c == ' ') else if (*c == ' ')
space = 1; space = 1;
@@ -82,7 +94,25 @@ int grub_create_loader_cmdline (int argc, char *argv[], char *buf, @@ -59,6 +71,17 @@ unsigned int grub_loader_cmdline_size (int argc, char *argv[])
return size;
}
+static grub_int32_t
+hex_to_int(char c)
+{
+ int i = c | 0x20;
+ if (i >= '0' && i < '9')
+ return i - '0';
+ if (i >= 'a' && i < 'f')
+ return i - ('a' - 10);
+ return -1;
+}
+
int grub_create_loader_cmdline (int argc, char *argv[], char *buf,
grub_size_t size)
{
@@ -82,7 +105,24 @@ int grub_create_loader_cmdline (int argc, char *argv[], char *buf,
while (*c) while (*c)
{ {
- if (*c == '\\' || *c == '\'' || *c == '"') - if (*c == '\\' || *c == '\'' || *c == '"')
+ if (*c == ' ') + if (*c < 0x21 || *c > 0x7e)
+ { + {
+ char int_to_hex[] = "0123456789abcdef";
+ *buf++ = '\\'; + *buf++ = '\\';
+ *buf++ = 'x'; + *buf++ = 'x';
+ *buf++ = '2'; + *buf++ = int_to_hex[(*c & 0xf0) >> 4];
+ *buf++ = '0'; + *buf++ = int_to_hex[*c & 0xf];
+ c++; + c++;
+ continue; + continue;
+ } + }
+ else if (*c == '\\' && *(c+1) == 'x' && + else if (c[0] == '\\' && c[1] == 'x' && is_hex(c[2]) && is_hex(c[3]))
+ is_hex(*(c+2)) && is_hex(*(c+3)))
+ { + {
+ *buf++ = *c++; + int a = (hex_to_int(c[2]) << 4) | (hex_to_int(c[3]) & 0xf);
+ *buf++ = *c++; + *buf++ = a & 0xff;
+ *buf++ = *c++; + c += 4;
+ *buf++ = *c++;
+ continue; + continue;
+ } + }
+ else if (*c == '\\' || *c == '\'' || *c == '"') + else if (*c == '\\' || *c == '\'' || *c == '"')
@ -103,10 +132,10 @@ index a702e64..c8605a7 100644
*buf++ = *c; *buf++ = *c;
diff --git a/grub-core/script/execute.c b/grub-core/script/execute.c diff --git a/grub-core/script/execute.c b/grub-core/script/execute.c
index b5e6eb0..c44eced 100644 index b5e6eb0..85a4bea 100644
--- a/grub-core/script/execute.c --- a/grub-core/script/execute.c
+++ b/grub-core/script/execute.c +++ b/grub-core/script/execute.c
@@ -52,6 +52,12 @@ static struct grub_script_scope *scope = 0; @@ -52,6 +52,23 @@ static struct grub_script_scope *scope = 0;
/* Wildcard translator for GRUB script. */ /* Wildcard translator for GRUB script. */
struct grub_script_wildcard_translator *grub_wildcard_translator; struct grub_script_wildcard_translator *grub_wildcard_translator;
@ -115,48 +144,56 @@ index b5e6eb0..c44eced 100644
+{ +{
+ return ((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F')); + return ((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F'));
+} +}
+
+static grub_int32_t
+hex_to_int(char c)
+{
+ int i = c | 0x20;
+ if (i >= '0' && i < '9')
+ return i - '0';
+ if (i >= 'a' && i < 'f')
+ return i - ('a' - 10);
+ return -1;
+}
+ +
static char* static char*
wildcard_escape (const char *s) wildcard_escape (const char *s)
{ {
@@ -68,7 +74,15 @@ wildcard_escape (const char *s) @@ -68,7 +85,16 @@ wildcard_escape (const char *s)
i = 0; i = 0;
while ((ch = *s++)) while ((ch = *s++))
{ {
- if (ch == '*' || ch == '\\' || ch == '?') - if (ch == '*' || ch == '\\' || ch == '?')
+ if (ch == '\\' && s[0] == 'x' && is_hex(s[1]) && is_hex(s[2])) + if (ch < 0x21 || ch > 0x7e)
+ { + {
+ p[i++] = ch; + char int_to_hex[] = "0123456789abcdef";
+ p[i++] = *s++; + p[i++] = '\\';
+ p[i++] = *s++; + p[i++] = 'x';
+ p[i++] = *s++; + p[i++] = int_to_hex[(ch & 0xf0) >> 4];
+ p[i++] = int_to_hex[ch & 0xf];
+ continue; + continue;
+ } + }
+ else if (ch == '*' || ch == '\\' || ch == '?') + else if (ch == '*' || ch == '\\' || ch == '?')
p[i++] = '\\'; p[i++] = '\\';
p[i++] = ch; p[i++] = ch;
} }
@@ -92,7 +106,14 @@ wildcard_unescape (const char *s) @@ -92,7 +118,14 @@ wildcard_unescape (const char *s)
i = 0; i = 0;
while ((ch = *s++)) while ((ch = *s++))
{ {
- if (ch == '\\') - if (ch == '\\')
+ if (ch == '\\' && s[0] == 'x' && is_hex(s[1]) && is_hex(s[2])) + if (ch == '\\' && s[0] == 'x' && is_hex(s[1]) && is_hex(s[2]))
+ { + {
+ p[i++] = '\\'; + int a = (hex_to_int(s[1]) << 4) | (hex_to_int(s[2]) & 0xf);
+ p[i++] = *s++; + p[i++] = a & 0xff;
+ p[i++] = *s++; + s += 3;
+ p[i++] = *s++; + continue;
+ } + }
+ else if (ch == '\\') + else if (ch == '\\')
p[i++] = *s++; p[i++] = *s++;
else else
p[i++] = ch; p[i++] = ch;
@@ -381,14 +402,24 @@ parse_string (const char *str, @@ -385,10 +418,20 @@ parse_string (const char *str,
int escaped = 0;
const char *optr;
for (ptr = str; ptr && *ptr; )
switch (*ptr) switch (*ptr)
{ {
case '\\': case '\\':
@ -164,12 +201,12 @@ index b5e6eb0..c44eced 100644
- if (!escaped && put) - if (!escaped && put)
- *((*put)++) = '\\'; - *((*put)++) = '\\';
- ptr++; - ptr++;
+ if (!escaped && put && *(ptr+1) == 'x' && is_hex(*(ptr+2)) && is_hex(*(ptr+3))) + if (!escaped && put && ptr[1] == 'x' && is_hex(ptr[2]) &&
+ is_hex(ptr[3]))
+ { + {
+ *((*put)++) = *ptr++; + int a = (hex_to_int(ptr[2]) << 4) | (hex_to_int(ptr[3]) & 0xf);
+ *((*put)++) = *ptr++; + *((*put)++) = a & 0xff;
+ *((*put)++) = *ptr++; + ptr += 4;
+ *((*put)++) = *ptr++;
+ } + }
+ else + else
+ { + {

View file

@ -90,7 +90,7 @@ BuildRequires: freetype-devel gettext-devel git
BuildRequires: texinfo BuildRequires: texinfo
BuildRequires: dejavu-sans-fonts BuildRequires: dejavu-sans-fonts
%ifarch %{efiarchs} %ifarch %{efiarchs}
BuildRequires: pesign >= 0.10-3 BuildRequires: pesign >= 0.99-8
%endif %endif
Requires: gettext os-prober which file system-logos Requires: gettext os-prober which file system-logos
@ -425,6 +425,9 @@ fi
%doc grub-%{tarversion}/themes/starfield/COPYING.CC-BY-SA-3.0 %doc grub-%{tarversion}/themes/starfield/COPYING.CC-BY-SA-3.0
%changelog %changelog
* Mon Oct 22 2012 Peter Jones <pjones@redhat.com> - 2.00-10
- Rebuild with newer pesign so we'll get signed with the final signing keys.
* Thu Oct 18 2012 Peter Jones <pjones@redhat.com> - 2.00-10 * Thu Oct 18 2012 Peter Jones <pjones@redhat.com> - 2.00-10
- Various PPC fixes. - Various PPC fixes.
- Fix crash fetching from http (gustavold, #860834) - Fix crash fetching from http (gustavold, #860834)