curl: fix build when proxy is not enabled in PACKAGECONFIG

Backport upstream patches to fix issues introduced by a backported CVE
patch. Fixes:

| ../../curl-7.75.0/lib/vtls/vtls.c: In function ‘Curl_ssl_addsessionid’:
| ../../curl-7.75.0/lib/vtls/vtls.c:508:14: error: ‘isProxy’ redeclared as different kind of symbol
|   508 |   const bool isProxy = FALSE;
|       |              ^~~~~~~
| ../../curl-7.75.0/lib/vtls/vtls.c:488:37: note: previous definition of ‘isProxy’ with type ‘_Bool’
|   488 |                                bool isProxy,
|       |                                     ^

(From OE-Core rev: e900a44e76dc2bb20ff725f24333c1c2b330bf41)

Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Anuj Mittal
2021-06-24 15:42:43 +08:00
committed by Richard Purdie
parent a175f86329
commit 57574529f8
3 changed files with 73 additions and 0 deletions

View File

@@ -0,0 +1,31 @@
From 2c26eeef12f0204fb85d6bf40b4e7a1e2ddcdf24 Mon Sep 17 00:00:00 2001
From: Daniel Stenberg <daniel@haxx.se>
Date: Mon, 29 Mar 2021 12:50:57 +0200
Subject: [PATCH] vtls: fix addsessionid for non-proxy builds
Follow-up to b09c8ee15771c61
Fixes #6812
Closes #6811
Upstream-Status: Backport [https://github.com/curl/curl/commit/2c26eeef12f0204fb85d6bf40b4e7a1e2ddcdf24]
Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
---
lib/vtls/vtls.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/lib/vtls/vtls.c b/lib/vtls/vtls.c
index 95fd6356285f..2e07df0a0462 100644
--- a/lib/vtls/vtls.c
+++ b/lib/vtls/vtls.c
@@ -504,11 +504,8 @@ CURLcode Curl_ssl_addsessionid(struct Curl_easy *data,
const char *hostname = isProxy ? conn->http_proxy.host.name :
conn->host.name;
#else
- /* proxy support disabled */
- const bool isProxy = FALSE;
struct ssl_primary_config * const ssl_config = &conn->ssl_config;
const char *hostname = conn->host.name;
- (void)sockindex;
#endif
(void)sockindex;
DEBUGASSERT(SSL_SET_OPTION(primary.sessionid));

View File

@@ -0,0 +1,40 @@
From b31d9ccfc2da288900e6857ad8d048c612328cac Mon Sep 17 00:00:00 2001
From: Jay Satiro <raysatiro@yahoo.com>
Date: Sun, 20 Jun 2021 16:42:58 -0400
Subject: [PATCH] vtls: fix warning due to function prototype mismatch
b09c8ee changed the function prototype. Caught by Visual Studio.
Upstream-Status: Backport [https://github.com/curl/curl/commit/b31d9ccfc2da288900e6857ad8d048c612328cac]
Signed-off-by: Anuj Mittal <anuj.mittal@intel.com>
---
lib/vtls/vtls.c | 2 +-
lib/vtls/vtls.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/vtls/vtls.c b/lib/vtls/vtls.c
index 82883c9c55e2..fe43703bf8b8 100644
--- a/lib/vtls/vtls.c
+++ b/lib/vtls/vtls.c
@@ -497,7 +497,7 @@ void Curl_ssl_delsessionid(struct Curl_easy *data, void *ssl_sessionid)
*/
CURLcode Curl_ssl_addsessionid(struct Curl_easy *data,
struct connectdata *conn,
- bool isProxy,
+ const bool isProxy,
void *ssl_sessionid,
size_t idsize,
int sockindex)
diff --git a/lib/vtls/vtls.h b/lib/vtls/vtls.h
index a0585c9cec4c..f1a9b8033ae5 100644
--- a/lib/vtls/vtls.h
+++ b/lib/vtls/vtls.h
@@ -247,7 +247,7 @@ void Curl_ssl_sessionid_unlock(struct Curl_easy *data);
*/
bool Curl_ssl_getsessionid(struct Curl_easy *data,
struct connectdata *conn,
- const bool isproxy,
+ const bool isProxy,
void **ssl_sessionid,
size_t *idsize, /* set 0 if unknown */
int sockindex);

View File

@@ -13,6 +13,8 @@ SRC_URI = "https://curl.haxx.se/download/curl-${PV}.tar.bz2 \
file://0001-replace-krb5-config-with-pkg-config.patch \
file://0001-vtls-add-isproxy-argument-to-Curl_ssl_get-addsession.patch \
file://0002-transfer-strip-credentials-from-the-auto-referer-hea.patch \
file://vtls-fix-addsessionid.patch \
file://vtls-fix-warning.patch \
"
SRC_URI[sha256sum] = "50552d4501c178e4cc68baaecc487f466a3d6d19bbf4e50a01869effb316d026"