Skip to content

Commit 1674d51

Browse files
Merge PR "[AUTO-CHERRYPICK] openssl: Fix check of unwrapped key size - branch main" microsoft#15076
Co-authored-by: corvus-callidus <[email protected]>
1 parent 4b2a5fb commit 1674d51

File tree

6 files changed

+62
-23
lines changed

6 files changed

+62
-23
lines changed
Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
From dfbaf161d8dafc1132dd88cd48ad990ed9b4c8ba Mon Sep 17 00:00:00 2001
2+
From: Viktor Dukhovni <[email protected]>
3+
Date: Thu, 11 Sep 2025 18:10:12 +0200
4+
Subject: [PATCH] kek_unwrap_key(): Fix incorrect check of unwrapped key size
5+
MIME-Version: 1.0
6+
Content-Type: text/plain; charset=UTF-8
7+
Content-Transfer-Encoding: 8bit
8+
9+
The check is off by 8 bytes so it is possible to overread by
10+
up to 8 bytes and overwrite up to 4 bytes.
11+
12+
Reviewed-by: Saša Nedvědický <[email protected]>
13+
Reviewed-by: Tomas Mraz <[email protected]>
14+
Reviewed-by: Neil Horman <[email protected]>
15+
(Merged from https://github.com/openssl/openssl/pull/6)
16+
---
17+
crypto/cms/cms_pwri.c | 2 +-
18+
1 file changed, 1 insertion(+), 1 deletion(-)
19+
20+
diff --git a/crypto/cms/cms_pwri.c b/crypto/cms/cms_pwri.c
21+
index d741488339..9f98840244 100644
22+
--- a/crypto/cms/cms_pwri.c
23+
+++ b/crypto/cms/cms_pwri.c
24+
@@ -215,7 +215,7 @@ static int kek_unwrap_key(unsigned char *out, size_t *outlen,
25+
/* Check byte failure */
26+
goto err;
27+
}
28+
- if (inlen < (size_t)(tmp[0] - 4)) {
29+
+ if (inlen < 4 + (size_t)tmp[0]) {
30+
/* Invalid length value */
31+
goto err;
32+
}
33+
--
34+
2.43.0

SPECS/openssl/openssl.spec

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
Summary: Utilities from the general purpose cryptography library with TLS implementation
55
Name: openssl
66
Version: 1.1.1k
7-
Release: 36%{?dist}
7+
Release: 37%{?dist}
88
License: OpenSSL
99
Vendor: Microsoft Corporation
1010
Distribution: Mariner
@@ -66,6 +66,8 @@ Patch42: openssl-1.1.1-Only-free-the-read-buffers-if-we-re-not-using-them
6666
Patch43: openssl-1.1.1-jitterentropy-fix-intermittent-fips-selftest-failure.patch
6767
Patch44: CVE-2024-5535.patch
6868
Patch45: openssl-1.1.1-Fix-timing-side-channel-in-ECDSA-signature-computation.patch
69+
Patch46: openssl-1.1.1-fix-incorrect-check-of-unwrapped-key-size.patch
70+
6971
BuildRequires: perl-Test-Warnings
7072
BuildRequires: perl-Text-Template
7173
BuildRequires: perl(FindBin)
@@ -329,6 +331,9 @@ rm -f %{buildroot}%{_sysconfdir}/pki/tls/ct_log_list.cnf.dist
329331
%postun libs -p /sbin/ldconfig
330332

331333
%changelog
334+
* Thu Nov 06 2025 Lynsey Rydberg <[email protected]> - 1.1.1k-37
335+
- Fix incorrect check of unwrapped key size
336+
332337
* Wed Mar 26 2025 Tobias Brick <[email protected]> - 1.1.1k-36
333338
- Fix timing side-channel in ECDSA signature computation.
334339

toolkit/resources/manifests/package/pkggen_core_aarch64.txt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -165,11 +165,11 @@ texinfo-6.8-1.cm2.aarch64.rpm
165165
gtk-doc-1.33.2-1.cm2.noarch.rpm
166166
autoconf-2.71-3.cm2.noarch.rpm
167167
automake-1.16.5-1.cm2.noarch.rpm
168-
openssl-1.1.1k-36.cm2.aarch64.rpm
169-
openssl-devel-1.1.1k-36.cm2.aarch64.rpm
170-
openssl-libs-1.1.1k-36.cm2.aarch64.rpm
171-
openssl-perl-1.1.1k-36.cm2.aarch64.rpm
172-
openssl-static-1.1.1k-36.cm2.aarch64.rpm
168+
openssl-1.1.1k-37.cm2.aarch64.rpm
169+
openssl-devel-1.1.1k-37.cm2.aarch64.rpm
170+
openssl-libs-1.1.1k-37.cm2.aarch64.rpm
171+
openssl-perl-1.1.1k-37.cm2.aarch64.rpm
172+
openssl-static-1.1.1k-37.cm2.aarch64.rpm
173173
libcap-2.60-4.cm2.aarch64.rpm
174174
libcap-devel-2.60-4.cm2.aarch64.rpm
175175
debugedit-5.0-2.cm2.aarch64.rpm

toolkit/resources/manifests/package/pkggen_core_x86_64.txt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -165,11 +165,11 @@ texinfo-6.8-1.cm2.x86_64.rpm
165165
gtk-doc-1.33.2-1.cm2.noarch.rpm
166166
autoconf-2.71-3.cm2.noarch.rpm
167167
automake-1.16.5-1.cm2.noarch.rpm
168-
openssl-1.1.1k-36.cm2.x86_64.rpm
169-
openssl-devel-1.1.1k-36.cm2.x86_64.rpm
170-
openssl-libs-1.1.1k-36.cm2.x86_64.rpm
171-
openssl-perl-1.1.1k-36.cm2.x86_64.rpm
172-
openssl-static-1.1.1k-36.cm2.x86_64.rpm
168+
openssl-1.1.1k-37.cm2.x86_64.rpm
169+
openssl-devel-1.1.1k-37.cm2.x86_64.rpm
170+
openssl-libs-1.1.1k-37.cm2.x86_64.rpm
171+
openssl-perl-1.1.1k-37.cm2.x86_64.rpm
172+
openssl-static-1.1.1k-37.cm2.x86_64.rpm
173173
libcap-2.60-4.cm2.x86_64.rpm
174174
libcap-devel-2.60-4.cm2.x86_64.rpm
175175
debugedit-5.0-2.cm2.x86_64.rpm

toolkit/resources/manifests/package/toolchain_aarch64.txt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -270,12 +270,12 @@ npth-1.6-4.cm2.aarch64.rpm
270270
npth-debuginfo-1.6-4.cm2.aarch64.rpm
271271
npth-devel-1.6-4.cm2.aarch64.rpm
272272
ntsysv-1.20-4.cm2.aarch64.rpm
273-
openssl-1.1.1k-36.cm2.aarch64.rpm
274-
openssl-debuginfo-1.1.1k-36.cm2.aarch64.rpm
275-
openssl-devel-1.1.1k-36.cm2.aarch64.rpm
276-
openssl-libs-1.1.1k-36.cm2.aarch64.rpm
277-
openssl-perl-1.1.1k-36.cm2.aarch64.rpm
278-
openssl-static-1.1.1k-36.cm2.aarch64.rpm
273+
openssl-1.1.1k-37.cm2.aarch64.rpm
274+
openssl-debuginfo-1.1.1k-37.cm2.aarch64.rpm
275+
openssl-devel-1.1.1k-37.cm2.aarch64.rpm
276+
openssl-libs-1.1.1k-37.cm2.aarch64.rpm
277+
openssl-perl-1.1.1k-37.cm2.aarch64.rpm
278+
openssl-static-1.1.1k-37.cm2.aarch64.rpm
279279
p11-kit-0.24.1-1.cm2.aarch64.rpm
280280
p11-kit-debuginfo-0.24.1-1.cm2.aarch64.rpm
281281
p11-kit-devel-0.24.1-1.cm2.aarch64.rpm

toolkit/resources/manifests/package/toolchain_x86_64.txt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -276,12 +276,12 @@ npth-1.6-4.cm2.x86_64.rpm
276276
npth-debuginfo-1.6-4.cm2.x86_64.rpm
277277
npth-devel-1.6-4.cm2.x86_64.rpm
278278
ntsysv-1.20-4.cm2.x86_64.rpm
279-
openssl-1.1.1k-36.cm2.x86_64.rpm
280-
openssl-debuginfo-1.1.1k-36.cm2.x86_64.rpm
281-
openssl-devel-1.1.1k-36.cm2.x86_64.rpm
282-
openssl-libs-1.1.1k-36.cm2.x86_64.rpm
283-
openssl-perl-1.1.1k-36.cm2.x86_64.rpm
284-
openssl-static-1.1.1k-36.cm2.x86_64.rpm
279+
openssl-1.1.1k-37.cm2.x86_64.rpm
280+
openssl-debuginfo-1.1.1k-37.cm2.x86_64.rpm
281+
openssl-devel-1.1.1k-37.cm2.x86_64.rpm
282+
openssl-libs-1.1.1k-37.cm2.x86_64.rpm
283+
openssl-perl-1.1.1k-37.cm2.x86_64.rpm
284+
openssl-static-1.1.1k-37.cm2.x86_64.rpm
285285
p11-kit-0.24.1-1.cm2.x86_64.rpm
286286
p11-kit-debuginfo-0.24.1-1.cm2.x86_64.rpm
287287
p11-kit-devel-0.24.1-1.cm2.x86_64.rpm

0 commit comments

Comments
 (0)