summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLee Chee Yang <chee.yang.lee@intel.com>2023-12-19 11:34:48 +0800
committerSteve Sakoman <steve@sakoman.com>2023-12-22 05:23:26 -1000
commit664c3401c57d4595bbd000a7d1dbf4427fcc1e88 (patch)
tree9f2e8e542d49e4de35ecde65c0275d258d7ac430
parent91bd70dbea333dcfedcb84db99b8298880a2cdd0 (diff)
downloadpoky-664c3401c57d4595bbd000a7d1dbf4427fcc1e88.tar.gz
curl: fix CVE-2023-46218
(From OE-Core rev: 000b2a380021f2c7fd102b6523a986c2e7ad20ae) Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com> Signed-off-by: Steve Sakoman <steve@sakoman.com>
-rw-r--r--meta/recipes-support/curl/curl/CVE-2023-46218.patch52
-rw-r--r--meta/recipes-support/curl/curl_8.4.0.bb1
2 files changed, 53 insertions, 0 deletions
diff --git a/meta/recipes-support/curl/curl/CVE-2023-46218.patch b/meta/recipes-support/curl/curl/CVE-2023-46218.patch
new file mode 100644
index 0000000000..de2f095664
--- /dev/null
+++ b/meta/recipes-support/curl/curl/CVE-2023-46218.patch
@@ -0,0 +1,52 @@
1CVE: CVE-2023-46218
2Upstream-Status: Backport [ https://github.com/curl/curl/commit/2b0994c29a721c91c57 ]
3Signed-off-by: Lee Chee Yang <chee.yang.lee@intel.com>
4
5From 2b0994c29a721c91c572cff7808c572a24d251eb Mon Sep 17 00:00:00 2001
6From: Daniel Stenberg <daniel@haxx.se>
7Date: Thu, 23 Nov 2023 08:15:47 +0100
8Subject: [PATCH] cookie: lowercase the domain names before PSL checks
9
10Reported-by: Harry Sintonen
11
12Closes #12387
13---
14 lib/cookie.c | 24 ++++++++++++++++--------
15 1 file changed, 16 insertions(+), 8 deletions(-)
16
17diff --git a/lib/cookie.c b/lib/cookie.c
18index 568cf537ad1b1f..9095cea3e97f22 100644
19--- a/lib/cookie.c
20+++ b/lib/cookie.c
21@@ -1027,15 +1027,23 @@ Curl_cookie_add(struct Curl_easy *data,
22 * dereference it.
23 */
24 if(data && (domain && co->domain && !Curl_host_is_ipnum(co->domain))) {
25- const psl_ctx_t *psl = Curl_psl_use(data);
26- int acceptable;
27-
28- if(psl) {
29- acceptable = psl_is_cookie_domain_acceptable(psl, domain, co->domain);
30- Curl_psl_release(data);
31+ bool acceptable = FALSE;
32+ char lcase[256];
33+ char lcookie[256];
34+ size_t dlen = strlen(domain);
35+ size_t clen = strlen(co->domain);
36+ if((dlen < sizeof(lcase)) && (clen < sizeof(lcookie))) {
37+ const psl_ctx_t *psl = Curl_psl_use(data);
38+ if(psl) {
39+ /* the PSL check requires lowercase domain name and pattern */
40+ Curl_strntolower(lcase, domain, dlen + 1);
41+ Curl_strntolower(lcookie, co->domain, clen + 1);
42+ acceptable = psl_is_cookie_domain_acceptable(psl, lcase, lcookie);
43+ Curl_psl_release(data);
44+ }
45+ else
46+ acceptable = !bad_domain(domain, strlen(domain));
47 }
48- else
49- acceptable = !bad_domain(domain, strlen(domain));
50
51 if(!acceptable) {
52 infof(data, "cookie '%s' dropped, domain '%s' must not "
diff --git a/meta/recipes-support/curl/curl_8.4.0.bb b/meta/recipes-support/curl/curl_8.4.0.bb
index 5f97730bf4..8f1ba52692 100644
--- a/meta/recipes-support/curl/curl_8.4.0.bb
+++ b/meta/recipes-support/curl/curl_8.4.0.bb
@@ -13,6 +13,7 @@ SRC_URI = " \
13 https://curl.se/download/${BP}.tar.xz \ 13 https://curl.se/download/${BP}.tar.xz \
14 file://run-ptest \ 14 file://run-ptest \
15 file://disable-tests \ 15 file://disable-tests \
16 file://CVE-2023-46218.patch \
16" 17"
17SRC_URI[sha256sum] = "16c62a9c4af0f703d28bda6d7bbf37ba47055ad3414d70dec63e2e6336f2a82d" 18SRC_URI[sha256sum] = "16c62a9c4af0f703d28bda6d7bbf37ba47055ad3414d70dec63e2e6336f2a82d"
18 19