diff options
author | Hauke Mehrtens <hauke@hauke-m.de> | 2019-05-17 22:40:26 +0200 |
---|---|---|
committer | Hauke Mehrtens <hauke@hauke-m.de> | 2019-05-30 12:15:20 +0200 |
commit | dc1b578a4cc1d7ec154a58baf3a813846c5adf9d (patch) | |
tree | e78010a927c172ef05b54d75bd178a451235a1bb /package/network/utils/curl/patches/403-CVE-2018-16840.patch | |
parent | 40ed8389efbb4011c83b6d343412a54634d0c731 (diff) | |
download | upstream-dc1b578a4cc1d7ec154a58baf3a813846c5adf9d.tar.gz upstream-dc1b578a4cc1d7ec154a58baf3a813846c5adf9d.tar.bz2 upstream-dc1b578a4cc1d7ec154a58baf3a813846c5adf9d.zip |
curl: Fix multiple security problems
This fixes the following security problems:
* CVE-2018-14618: NTLM password overflow via integer overflow
* CVE-2018-16839: SASL password overflow via integer overflow
* CVE-2018-16840: use-after-free in handle close
* CVE-2018-16842: warning message out-of-buffer read
* CVE-2019-3823: SMTP end-of-response out-of-bounds read
* CVE-2019-3822: NTLMv2 type-3 header stack buffer overflow
* CVE-2018-16890: NTLM type-2 out-of-bounds buffer read
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
Diffstat (limited to 'package/network/utils/curl/patches/403-CVE-2018-16840.patch')
-rw-r--r-- | package/network/utils/curl/patches/403-CVE-2018-16840.patch | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/package/network/utils/curl/patches/403-CVE-2018-16840.patch b/package/network/utils/curl/patches/403-CVE-2018-16840.patch new file mode 100644 index 0000000000..00a36f6f19 --- /dev/null +++ b/package/network/utils/curl/patches/403-CVE-2018-16840.patch @@ -0,0 +1,31 @@ +From 81d135d67155c5295b1033679c606165d4e28f3f Mon Sep 17 00:00:00 2001 +From: Daniel Stenberg <daniel@haxx.se> +Date: Thu, 18 Oct 2018 15:07:15 +0200 +Subject: [PATCH] Curl_close: clear data->multi_easy on free to avoid + use-after-free + +Regression from b46cfbc068 (7.59.0) +CVE-2018-16840 +Reported-by: Brian Carpenter (Geeknik Labs) + +Bug: https://curl.haxx.se/docs/CVE-2018-16840.html +--- + lib/url.c | 4 +++- + 1 file changed, 3 insertions(+), 1 deletion(-) + +--- a/lib/url.c ++++ b/lib/url.c +@@ -320,10 +320,12 @@ CURLcode Curl_close(struct Curl_easy *da + and detach this handle from there. */ + curl_multi_remove_handle(data->multi, data); + +- if(data->multi_easy) ++ if(data->multi_easy) { + /* when curl_easy_perform() is used, it creates its own multi handle to + use and this is the one */ + curl_multi_cleanup(data->multi_easy); ++ data->multi_easy = NULL; ++ } + + /* Destroy the timeout list that is held in the easy handle. It is + /normally/ done by curl_multi_remove_handle() but this is "just in |