← Commit list

7aaf9a3152

Daniel Stenberg
Daniel Stenberg
2025-11-12
SUCCESS Duration: 149s

Subject

http: replace atoi use in Curl_http_follow with curlx_str_number

Binary

No binary (build failed)

Architecture & Compiler Flags

Architecture: x86_64-linux-gnu Compiler: gcc (Ubuntu 11.4.0-1ubuntu1~22.04.3) 11.4.0

—

Build Log

(log yok)

Commit Diff GitHub →

lib/http.c+6 -2
@@ -1302,7 +1302,6 @@ CURLcode Curl_http_follow(struct Curl_easy *data, const char *newurl,
/* Clear auth if this redirects to a different port number or protocol,
unless permitted */
if(!data->set.allow_auth_to_other_hosts && (type != FOLLOW_FAKE)) {
- char *portnum;
int port;
bool clear = FALSE;
@@ -1310,13 +1309,18 @@ CURLcode Curl_http_follow(struct Curl_easy *data, const char *newurl,
/* a custom port is used */
port = (int)data->set.use_port;
else {
+ curl_off_t value;
+ char *portnum;
+ const char *p;
uc = curl_url_get(data->state.uh, CURLUPART_PORT, &portnum,
CURLU_DEFAULT_PORT);
if(uc) {
free(follow_url);
return Curl_uc_to_curlcode(uc);
}
- port = atoi(portnum);
+ p = portnum;
+ curlx_str_number(&p, &value, 0xffff);
+ port = (int)value;
free(portnum);
}
if(port != data->info.conn_remote_port) {