Skip to content

Commit c807604

Browse files
committed
Merge branch 'jk/curl-easy-setopt-typefix' into js/curl-easy-setopt-typefix
* jk/curl-easy-setopt-typefix: curl: fix symbolic constant typechecks with curl_easy_setopt() curl: fix integer variable typechecks with curl_easy_setopt() curl: fix integer constant typechecks with curl_easy_setopt()
2 parents 0d42fbd + 4558c8f commit c807604

File tree

4 files changed

+21
-21
lines changed

4 files changed

+21
-21
lines changed

http-push.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,7 @@ static char *xml_entities(const char *s)
195195
static void curl_setup_http_get(CURL *curl, const char *url,
196196
const char *custom_req)
197197
{
198-
curl_easy_setopt(curl, CURLOPT_HTTPGET, 1);
198+
curl_easy_setopt(curl, CURLOPT_HTTPGET, 1L);
199199
curl_easy_setopt(curl, CURLOPT_URL, url);
200200
curl_easy_setopt(curl, CURLOPT_CUSTOMREQUEST, custom_req);
201201
curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, fwrite_null);

http.c

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1019,13 +1019,13 @@ static CURL *get_curl_handle(void)
10191019
die("curl_easy_init failed");
10201020

10211021
if (!curl_ssl_verify) {
1022-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 0);
1023-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 0);
1022+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 0L);
1023+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 0L);
10241024
} else {
10251025
/* Verify authenticity of the peer's certificate */
1026-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 1);
1026+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 1L);
10271027
/* The name in the cert must match whom we tried to connect */
1028-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 2);
1028+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 2L);
10291029
}
10301030

10311031
if (curl_http_version) {
@@ -1057,7 +1057,7 @@ static CURL *get_curl_handle(void)
10571057

10581058
if (http_ssl_backend && !strcmp("schannel", http_ssl_backend) &&
10591059
!http_schannel_check_revoke) {
1060-
curl_easy_setopt(result, CURLOPT_SSL_OPTIONS, CURLSSLOPT_NO_REVOKE);
1060+
curl_easy_setopt(result, CURLOPT_SSL_OPTIONS, (long)CURLSSLOPT_NO_REVOKE);
10611061
}
10621062

10631063
if (http_proactive_auth != PROACTIVE_AUTH_NONE)
@@ -1117,8 +1117,8 @@ static CURL *get_curl_handle(void)
11171117
curl_low_speed_time);
11181118
}
11191119

1120-
curl_easy_setopt(result, CURLOPT_MAXREDIRS, 20);
1121-
curl_easy_setopt(result, CURLOPT_POSTREDIR, CURL_REDIR_POST_ALL);
1120+
curl_easy_setopt(result, CURLOPT_MAXREDIRS, 20L);
1121+
curl_easy_setopt(result, CURLOPT_POSTREDIR, (long)CURL_REDIR_POST_ALL);
11221122

11231123
#ifdef GIT_CURL_HAVE_CURLOPT_PROTOCOLS_STR
11241124
{
@@ -1151,7 +1151,7 @@ static CURL *get_curl_handle(void)
11511151
user_agent ? user_agent : git_user_agent());
11521152

11531153
if (curl_ftp_no_epsv)
1154-
curl_easy_setopt(result, CURLOPT_FTP_USE_EPSV, 0);
1154+
curl_easy_setopt(result, CURLOPT_FTP_USE_EPSV, 0L);
11551155

11561156
if (curl_ssl_try)
11571157
curl_easy_setopt(result, CURLOPT_USE_SSL, CURLUSESSL_TRY);
@@ -1193,18 +1193,18 @@ static CURL *get_curl_handle(void)
11931193

11941194
if (starts_with(curl_http_proxy, "socks5h"))
11951195
curl_easy_setopt(result,
1196-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS5_HOSTNAME);
1196+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS5_HOSTNAME);
11971197
else if (starts_with(curl_http_proxy, "socks5"))
11981198
curl_easy_setopt(result,
1199-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS5);
1199+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS5);
12001200
else if (starts_with(curl_http_proxy, "socks4a"))
12011201
curl_easy_setopt(result,
1202-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS4A);
1202+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS4A);
12031203
else if (starts_with(curl_http_proxy, "socks"))
12041204
curl_easy_setopt(result,
1205-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS4);
1205+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS4);
12061206
else if (starts_with(curl_http_proxy, "https")) {
1207-
curl_easy_setopt(result, CURLOPT_PROXYTYPE, CURLPROXY_HTTPS);
1207+
curl_easy_setopt(result, CURLOPT_PROXYTYPE, (long)CURLPROXY_HTTPS);
12081208

12091209
if (http_proxy_ssl_cert)
12101210
curl_easy_setopt(result, CURLOPT_PROXY_SSLCERT, http_proxy_ssl_cert);
@@ -1254,7 +1254,7 @@ static CURL *get_curl_handle(void)
12541254
}
12551255
init_curl_proxy_auth(result);
12561256

1257-
curl_easy_setopt(result, CURLOPT_TCP_KEEPALIVE, 1);
1257+
curl_easy_setopt(result, CURLOPT_TCP_KEEPALIVE, 1L);
12581258

12591259
if (curl_tcp_keepidle > -1)
12601260
curl_easy_setopt(result, CURLOPT_TCP_KEEPIDLE,

imap-send.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1420,7 +1420,7 @@ static CURL *setup_curl(struct imap_server_conf *srvc, struct credential *cred)
14201420

14211421
curl_easy_setopt(curl, CURLOPT_URL, path.buf);
14221422
strbuf_release(&path);
1423-
curl_easy_setopt(curl, CURLOPT_PORT, srvc->port);
1423+
curl_easy_setopt(curl, CURLOPT_PORT, (long)srvc->port);
14241424

14251425
if (srvc->auth_method) {
14261426
struct strbuf auth = STRBUF_INIT;
@@ -1433,8 +1433,8 @@ static CURL *setup_curl(struct imap_server_conf *srvc, struct credential *cred)
14331433
if (!srvc->use_ssl)
14341434
curl_easy_setopt(curl, CURLOPT_USE_SSL, (long)CURLUSESSL_TRY);
14351435

1436-
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYPEER, srvc->ssl_verify);
1437-
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYHOST, srvc->ssl_verify);
1436+
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYPEER, (long)srvc->ssl_verify);
1437+
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYHOST, (long)srvc->ssl_verify);
14381438

14391439
curl_easy_setopt(curl, CURLOPT_READFUNCTION, fread_buffer);
14401440

remote-curl.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -877,12 +877,12 @@ static int probe_rpc(struct rpc_state *rpc, struct slot_results *results)
877877
headers = curl_slist_append(headers, rpc->hdr_content_type);
878878
headers = curl_slist_append(headers, rpc->hdr_accept);
879879

880-
curl_easy_setopt(slot->curl, CURLOPT_NOBODY, 0);
881-
curl_easy_setopt(slot->curl, CURLOPT_POST, 1);
880+
curl_easy_setopt(slot->curl, CURLOPT_NOBODY, 0L);
881+
curl_easy_setopt(slot->curl, CURLOPT_POST, 1L);
882882
curl_easy_setopt(slot->curl, CURLOPT_URL, rpc->service_url);
883883
curl_easy_setopt(slot->curl, CURLOPT_ENCODING, NULL);
884884
curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDS, "0000");
885-
curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE, 4);
885+
curl_easy_setopt(slot->curl, CURLOPT_POSTFIELDSIZE, 4L);
886886
curl_easy_setopt(slot->curl, CURLOPT_HTTPHEADER, headers);
887887
curl_easy_setopt(slot->curl, CURLOPT_WRITEFUNCTION, fwrite_buffer);
888888
curl_easy_setopt(slot->curl, CURLOPT_WRITEDATA, &buf);

0 commit comments

Comments
 (0)