Skip to contentSkip to conversation

libcurl: security bump to version 7.42.0

PATCH New Gustavo Zacarias 2 participants 2 messages Apr 23, 2015

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 libcurl: security bump to version 7.42.0

Fixes:
CVE-2015-3144 - host name out of boundary memory access CVE-2015-3145 - cookie parser out of boundary memory access CVE-2015-3148 - Negotiate not treated as connection-oriented CVE-2015-3143 - Re-using authenticated connection when unauthenticated

---
 ...1-connectionexists-fix-build-without-NTLM.patch | 54 ++++++++++++++++++++++
 ...connectionexists-follow-up-to-fd9d3a1ef1f.patch | 48 +++++++++++++++++++
 package/libcurl/libcurl.hash                       |  2 +-
 package/libcurl/libcurl.mk                         |  2 +-
 4 files changed, 104 insertions(+), 2 deletions(-)
 create mode 100644 package/libcurl/0001-connectionexists-fix-build-without-NTLM.patch
 create mode 100644 package/libcurl/0002-connectionexists-follow-up-to-fd9d3a1ef1f.patch
package/libcurl/0001-connectionexists-fix-build-without-NTLM.patch new file +54 -0
new file mode 100644
index 0000000..4f91372
@@ -0,0 +1,54 @@
+From fd9d3a1ef1f7b1cb5812d04bad07818efc6f3b3a Mon Sep 17 00:00:00 2001
+From: Daniel Stenberg <[email protected]>
+Date: Wed, 22 Apr 2015 13:31:35 +0200
+Subject: [PATCH 1/2] connectionexists: fix build without NTLM
+
+Do not access NTLM-specific struct fields when built without NTLM
+enabled!
+
+bug: http://curl.haxx.se/?i=231
+Reported-by: Patrick Rapin
+Signed-off-by: Gustavo Zacarias <[email protected]>
+---
+ lib/url.c | 6 ++++--
+ 1 file changed, 4 insertions(+), 2 deletions(-)
+
+diff --git a/lib/url.c b/lib/url.c
+index f033dbc..93f15f1 100644
+--- a/lib/url.c
++++ b/lib/url.c
+@@ -3069,9 +3069,11 @@ ConnectionExists(struct SessionHandle *data,
+   struct connectdata *check;
+   struct connectdata *chosen = 0;
+   bool canPipeline = IsPipeliningPossible(data, needle);
++#ifdef USE_NTLM
+   bool wantNTLMhttp = ((data->state.authhost.want & CURLAUTH_NTLM) ||
+                        (data->state.authhost.want & CURLAUTH_NTLM_WB)) &&
+     (needle->handler->protocol & PROTO_FAMILY_HTTP) ? TRUE : FALSE;
++#endif
+   struct connectbundle *bundle;
+ 
+   *force_reuse = FALSE;
+@@ -3208,6 +3210,7 @@ ConnectionExists(struct SessionHandle *data,
+           continue;
+       }
+ 
++#if defined(USE_NTLM)
+       if((!(needle->handler->flags & PROTOPT_CREDSPERREQUEST)) ||
+          (wantNTLMhttp || check->ntlm.state != NTLMSTATE_NONE)) {
+         /* This protocol requires credentials per connection or is HTTP+NTLM,
+@@ -3217,10 +3220,9 @@ ConnectionExists(struct SessionHandle *data,
+           /* one of them was different */
+           continue;
+         }
+-#if defined(USE_NTLM)
+         credentialsMatch = TRUE;
+-#endif
+       }
++#endif
+ 
+       if(!needle->bits.httpproxy || needle->handler->flags&PROTOPT_SSL ||
+          (needle->bits.httpproxy && check->bits.httpproxy &&
+-- 
+2.0.5
+
package/libcurl/0002-connectionexists-follow-up-to-fd9d3a1ef1f.patch new file +48 -0

48 lines not on this page · raw message

package/libcurl/libcurl.hash +1 -1
index 3b00f0d..e2bd83d 100644
@@ -1,2 +1,2 @@
 # Locally calculated after checking pgp signature
-sha256	9f8b546bdc5c57d959151acae7ce6610fe929d82b8d0fc5b25a3a2296e5f8bea	curl-7.41.0.tar.bz2
+sha256	32557d68542f5c6cc8437b5b8a945857b4c5c6b6276da909e35b783d1d66d08f	curl-7.42.0.tar.bz2
package/libcurl/libcurl.mk +1 -1
index 69cd8df..acb2b42 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-LIBCURL_VERSION = 7.41.0
+LIBCURL_VERSION = 7.42.0
 LIBCURL_SOURCE = curl-$(LIBCURL_VERSION).tar.bz2
 LIBCURL_SITE = http://curl.haxx.se/download
 LIBCURL_DEPENDENCIES = host-pkgconf \
Signature · 1 line
2.0.5
#1 lore raw

PATCH 1/1 libcurl: security bump to version 7.42.0

Dear Gustavo Zacarias,

Quoted 7 lines · Gustavo Zacarias

Fixes:
CVE-2015-3144 - host name out of boundary memory access CVE-2015-3145 - cookie parser out of boundary memory access CVE-2015-3148 - Negotiate not treated as connection-oriented CVE-2015-3143 - Re-using authenticated connection when unauthenticated

Quoted 8 lines
---
 ...1-connectionexists-fix-build-without-NTLM.patch | 54 ++++++++++++++++++++++
 ...connectionexists-follow-up-to-fd9d3a1ef1f.patch | 48 +++++++++++++++++++
 package/libcurl/libcurl.hash                       |  2 +-
 package/libcurl/libcurl.mk                         |  2 +-
 4 files changed, 104 insertions(+), 2 deletions(-)
 create mode 100644 package/libcurl/0001-connectionexists-fix-build-without-NTLM.patch
 create mode 100644 package/libcurl/0002-connectionexists-follow-up-to-fd9d3a1ef1f.patch

Applied, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com