Skip to content

Commit 3fb69b5

Browse files
gitsterdscho
authored andcommitted
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 cca1f38 + 018cbc4 commit 3fb69b5

File tree

4 files changed

+20
-20
lines changed

4 files changed

+20
-20
lines changed

http-push.c

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

http.c

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -731,7 +731,7 @@ static int has_proxy_cert_password(void)
731731

732732
static void set_curl_keepalive(CURL *c)
733733
{
734-
curl_easy_setopt(c, CURLOPT_TCP_KEEPALIVE, 1);
734+
curl_easy_setopt(c, CURLOPT_TCP_KEEPALIVE, 1L);
735735
}
736736

737737
/* Return 1 if redactions have been made, 0 otherwise. */
@@ -1032,13 +1032,13 @@ static CURL *get_curl_handle(void)
10321032
die("curl_easy_init failed");
10331033

10341034
if (!curl_ssl_verify) {
1035-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 0);
1036-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 0);
1035+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 0L);
1036+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 0L);
10371037
} else {
10381038
/* Verify authenticity of the peer's certificate */
1039-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 1);
1039+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYPEER, 1L);
10401040
/* The name in the cert must match whom we tried to connect */
1041-
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 2);
1041+
curl_easy_setopt(result, CURLOPT_SSL_VERIFYHOST, 2L);
10421042
}
10431043

10441044
if (curl_http_version) {
@@ -1141,8 +1141,8 @@ static CURL *get_curl_handle(void)
11411141
curl_low_speed_time);
11421142
}
11431143

1144-
curl_easy_setopt(result, CURLOPT_MAXREDIRS, 20);
1145-
curl_easy_setopt(result, CURLOPT_POSTREDIR, CURL_REDIR_POST_ALL);
1144+
curl_easy_setopt(result, CURLOPT_MAXREDIRS, 20L);
1145+
curl_easy_setopt(result, CURLOPT_POSTREDIR, (long)CURL_REDIR_POST_ALL);
11461146

11471147
#ifdef GIT_CURL_HAVE_CURLOPT_PROTOCOLS_STR
11481148
{
@@ -1175,7 +1175,7 @@ static CURL *get_curl_handle(void)
11751175
user_agent ? user_agent : git_user_agent());
11761176

11771177
if (curl_ftp_no_epsv)
1178-
curl_easy_setopt(result, CURLOPT_FTP_USE_EPSV, 0);
1178+
curl_easy_setopt(result, CURLOPT_FTP_USE_EPSV, 0L);
11791179

11801180
if (curl_ssl_try)
11811181
curl_easy_setopt(result, CURLOPT_USE_SSL, CURLUSESSL_TRY);
@@ -1217,18 +1217,18 @@ static CURL *get_curl_handle(void)
12171217

12181218
if (starts_with(curl_http_proxy, "socks5h"))
12191219
curl_easy_setopt(result,
1220-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS5_HOSTNAME);
1220+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS5_HOSTNAME);
12211221
else if (starts_with(curl_http_proxy, "socks5"))
12221222
curl_easy_setopt(result,
1223-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS5);
1223+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS5);
12241224
else if (starts_with(curl_http_proxy, "socks4a"))
12251225
curl_easy_setopt(result,
1226-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS4A);
1226+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS4A);
12271227
else if (starts_with(curl_http_proxy, "socks"))
12281228
curl_easy_setopt(result,
1229-
CURLOPT_PROXYTYPE, CURLPROXY_SOCKS4);
1229+
CURLOPT_PROXYTYPE, (long)CURLPROXY_SOCKS4);
12301230
else if (starts_with(curl_http_proxy, "https")) {
1231-
curl_easy_setopt(result, CURLOPT_PROXYTYPE, CURLPROXY_HTTPS);
1231+
curl_easy_setopt(result, CURLOPT_PROXYTYPE, (long)CURLPROXY_HTTPS);
12321232

12331233
if (http_proxy_ssl_cert)
12341234
curl_easy_setopt(result, CURLOPT_PROXY_SSLCERT, http_proxy_ssl_cert);

imap-send.c

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

14191419
curl_easy_setopt(curl, CURLOPT_URL, path.buf);
14201420
strbuf_release(&path);
1421-
curl_easy_setopt(curl, CURLOPT_PORT, srvc->port);
1421+
curl_easy_setopt(curl, CURLOPT_PORT, (long)srvc->port);
14221422

14231423
if (srvc->auth_method) {
14241424
struct strbuf auth = STRBUF_INIT;
@@ -1431,8 +1431,8 @@ static CURL *setup_curl(struct imap_server_conf *srvc, struct credential *cred)
14311431
if (!srvc->use_ssl)
14321432
curl_easy_setopt(curl, CURLOPT_USE_SSL, (long)CURLUSESSL_TRY);
14331433

1434-
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYPEER, srvc->ssl_verify);
1435-
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYHOST, srvc->ssl_verify);
1434+
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYPEER, (long)srvc->ssl_verify);
1435+
curl_easy_setopt(curl, CURLOPT_SSL_VERIFYHOST, (long)srvc->ssl_verify);
14361436

14371437
curl_easy_setopt(curl, CURLOPT_READFUNCTION, fread_buffer);
14381438

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)