summaryrefslogtreecommitdiffstats
path: root/meta/recipes-bsp/grub/files/CVE-2024-56738.patch
diff options
context:
space:
mode:
Diffstat (limited to 'meta/recipes-bsp/grub/files/CVE-2024-56738.patch')
-rw-r--r--meta/recipes-bsp/grub/files/CVE-2024-56738.patch75
1 files changed, 75 insertions, 0 deletions
diff --git a/meta/recipes-bsp/grub/files/CVE-2024-56738.patch b/meta/recipes-bsp/grub/files/CVE-2024-56738.patch
new file mode 100644
index 0000000000..c7b64aa6ed
--- /dev/null
+++ b/meta/recipes-bsp/grub/files/CVE-2024-56738.patch
@@ -0,0 +1,75 @@
1From 4cef2fc7308b2132317ad166939994f098b41561 Mon Sep 17 00:00:00 2001
2From: Ross Burton <ross.burton@arm.com>
3Date: Tue, 9 Sep 2025 14:23:14 +0100
4Subject: [PATCH] CVE-2024-56738
5
6Backport an algorithmic change to grub_crypto_memcmp() so that it completes in
7constant time and thus isn't susceptible to side-channel attacks.
8
9This is a partial backport of grub 0739d24cd
10("libgcrypt: Adjust import script, definitions and API users for libgcrypt 1.11")
11
12CVE: CVE-2024-56738
13Upstream-Status: Backport [0739d24cd]
14Signed-off-by: Ross Burton <ross.burton@arm.com>
15---
16 grub-core/lib/crypto.c | 23 ++++++++++++++++-------
17 include/grub/crypto.h | 2 +-
18 2 files changed, 17 insertions(+), 8 deletions(-)
19
20diff --git a/grub-core/lib/crypto.c b/grub-core/lib/crypto.c
21index 396f76410..19db7870a 100644
22--- a/grub-core/lib/crypto.c
23+++ b/grub-core/lib/crypto.c
24@@ -433,19 +433,28 @@ grub_crypto_gcry_error (gcry_err_code_t in)
25 return GRUB_ACCESS_DENIED;
26 }
27
28+/*
29+ * Compare byte arrays of length LEN, return 1 if it's not same,
30+ * 0, otherwise.
31+ */
32 int
33-grub_crypto_memcmp (const void *a, const void *b, grub_size_t n)
34+grub_crypto_memcmp (const void *b1, const void *b2, grub_size_t len)
35 {
36- register grub_size_t counter = 0;
37- const grub_uint8_t *pa, *pb;
38+ const grub_uint8_t *a = b1;
39+ const grub_uint8_t *b = b2;
40+ int ab, ba;
41+ grub_size_t i;
42
43- for (pa = a, pb = b; n; pa++, pb++, n--)
44+ /* Constant-time compare. */
45+ for (i = 0, ab = 0, ba = 0; i < len; i++)
46 {
47- if (*pa != *pb)
48- counter++;
49+ /* If a[i] != b[i], either ab or ba will be negative. */
50+ ab |= a[i] - b[i];
51+ ba |= b[i] - a[i];
52 }
53
54- return !!counter;
55+ /* 'ab | ba' is negative when buffers are not equal, extract sign bit. */
56+ return ((unsigned int)(ab | ba) >> (sizeof(unsigned int) * 8 - 1)) & 1;
57 }
58
59 #ifndef GRUB_UTIL
60diff --git a/include/grub/crypto.h b/include/grub/crypto.h
61index 31c87c302..20ad4c5f7 100644
62--- a/include/grub/crypto.h
63+++ b/include/grub/crypto.h
64@@ -393,7 +393,7 @@ grub_crypto_pbkdf2 (const struct gcry_md_spec *md,
65 grub_uint8_t *DK, grub_size_t dkLen);
66
67 int
68-grub_crypto_memcmp (const void *a, const void *b, grub_size_t n);
69+grub_crypto_memcmp (const void *b1, const void *b2, grub_size_t len);
70
71 int
72 grub_password_get (char buf[], unsigned buf_size);
73--
742.43.0
75