diff options
Diffstat (limited to 'meta/recipes-support/libksba/libksba/CVE-2022-3515.patch')
-rw-r--r-- | meta/recipes-support/libksba/libksba/CVE-2022-3515.patch | 47 |
1 files changed, 47 insertions, 0 deletions
diff --git a/meta/recipes-support/libksba/libksba/CVE-2022-3515.patch b/meta/recipes-support/libksba/libksba/CVE-2022-3515.patch new file mode 100644 index 0000000000..ff9f2f9275 --- /dev/null +++ b/meta/recipes-support/libksba/libksba/CVE-2022-3515.patch | |||
@@ -0,0 +1,47 @@ | |||
1 | From 4b7d9cd4a018898d7714ce06f3faf2626c14582b Mon Sep 17 00:00:00 2001 | ||
2 | From: Werner Koch <wk@gnupg.org> | ||
3 | Date: Wed, 5 Oct 2022 14:19:06 +0200 | ||
4 | Subject: [PATCH] Detect a possible overflow directly in the TLV parser. | ||
5 | |||
6 | * src/ber-help.c (_ksba_ber_read_tl): Check for overflow of a commonly | ||
7 | used sum. | ||
8 | -- | ||
9 | |||
10 | It is quite common to have checks like | ||
11 | |||
12 | if (ti.nhdr + ti.length >= DIM(tmpbuf)) | ||
13 | return gpg_error (GPG_ERR_TOO_LARGE); | ||
14 | |||
15 | This patch detects possible integer overflows immmediately when | ||
16 | creating the TI object. | ||
17 | |||
18 | Reported-by: ZDI-CAN-18927, ZDI-CAN-18928, ZDI-CAN-18929 | ||
19 | |||
20 | |||
21 | Upstream-Status: Backport [https://git.gnupg.org/cgi-bin/gitweb.cgi?p=libksba.git;a=patch;h=4b7d9cd4a018898d7714ce06f3faf2626c14582b] | ||
22 | CVE: CVE-2022-3515 | ||
23 | Signed-off-by: Chee Yang Lee <chee.yang.lee@intel.com> | ||
24 | --- | ||
25 | src/ber-help.c | 6 ++++++ | ||
26 | 1 file changed, 6 insertions(+) | ||
27 | |||
28 | diff --git a/src/ber-help.c b/src/ber-help.c | ||
29 | index 81c31ed..56efb6a 100644 | ||
30 | --- a/src/ber-help.c | ||
31 | +++ b/src/ber-help.c | ||
32 | @@ -182,6 +182,12 @@ _ksba_ber_read_tl (ksba_reader_t reader, struct tag_info *ti) | ||
33 | ti->length = len; | ||
34 | } | ||
35 | |||
36 | + if (ti->length > ti->nhdr && (ti->nhdr + ti->length) < ti->length) | ||
37 | + { | ||
38 | + ti->err_string = "header+length would overflow"; | ||
39 | + return gpg_error (GPG_ERR_EOVERFLOW); | ||
40 | + } | ||
41 | + | ||
42 | /* Without this kludge some example certs can't be parsed */ | ||
43 | if (ti->class == CLASS_UNIVERSAL && !ti->tag) | ||
44 | ti->length = 0; | ||
45 | -- | ||
46 | 2.11.0 | ||
47 | |||