curl: CVE-2016-8617

OOB write via unchecked multiplication

Affected versions: curl 7.1 to and including 7.50.3

Reference:
https://curl.haxx.se/docs/adv_20161102C.html

(From OE-Core rev: 82415212303d75ca9a6f15a9abda42c9675efde4)

Signed-off-by: Sona Sarmadi <sona.sarmadi@enea.com>
Signed-off-by: Armin Kuster <akuster808@gmail.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Sona Sarmadi 2016-11-15 10:08:12 +01:00 committed by Richard Purdie
parent 9a72d46aed
commit eb0dff0c98
2 changed files with 29 additions and 0 deletions

View File

@ -0,0 +1,28 @@
From efd24d57426bd77c9b5860e6b297904703750412 Mon Sep 17 00:00:00 2001
From: Daniel Stenberg <daniel@haxx.se>
Date: Wed, 28 Sep 2016 00:05:12 +0200
Subject: [PATCH] base64: check for integer overflow on large input
CVE: CVE-2016-8617
Upstream-Status: Backport
Bug: https://curl.haxx.se/docs/adv_20161102C.html
Reported-by: Cure53
Signed-off-by: Sona Sarmadi <sona.sarmadi@enea.com>
---
diff -ruN a/lib/base64.c b/lib/base64.c
--- a/lib/base64.c 2016-02-03 00:02:43.000000000 +0100
+++ b/lib/base64.c 2016-11-07 09:22:07.918167530 +0100
@@ -190,6 +190,11 @@
if(0 == insize)
insize = strlen(indata);
+#if SIZEOF_SIZE_T == 4
+ if(insize > UINT_MAX/4)
+ return CURLE_OUT_OF_MEMORY;
+#endif
+
base64data = output = malloc(insize*4/3+4);
if(NULL == output)
return CURLE_OUT_OF_MEMORY;

View File

@ -17,6 +17,7 @@ SRC_URI += " file://configure_ac.patch \
file://CVE-2016-7141.patch \
file://CVE-2016-8615.patch \
file://CVE-2016-8616.patch \
file://CVE-2016-8617.patch \
"
SRC_URI[md5sum] = "9ea3123449439bbd960cd25cf98796fb"