diff options
author | Daniel Stenberg <daniel@haxx.se> | 2019-07-30 11:02:03 +0200 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2019-07-30 11:02:03 +0200 |
commit | a405fb6748a96d7f6935efcf0b50900b9274ab00 (patch) | |
tree | 50f36fb7de90e5550646cc95864d8d9c9aa222a4 /lib/url.c | |
parent | 041690aadb1357775ff06c5bf827a98585627c76 (diff) | |
download | curl-bagder/remove-numsocks.tar.gz |
cleanup: remove the 'numsocks' argument used in many placesbagder/remove-numsocks
It was used (intended) to pass in the size of the 'socks' array that is
also passed to these functions, but was rarely actually checked/used and
the array is defined to a fixed size of MAX_SOCKSPEREASYHANDLE entries
that should be used instead.
Diffstat (limited to 'lib/url.c')
-rw-r--r-- | lib/url.c | 10 |
1 files changed, 4 insertions, 6 deletions
@@ -1440,11 +1440,10 @@ void Curl_verboseconnect(struct connectdata *conn) #endif int Curl_protocol_getsock(struct connectdata *conn, - curl_socket_t *socks, - int numsocks) + curl_socket_t *socks) { if(conn->handler->proto_getsock) - return conn->handler->proto_getsock(conn, socks, numsocks); + return conn->handler->proto_getsock(conn, socks); /* Backup getsock logic. Since there is a live socket in use, we must wait for it or it will be removed from watching when the multi_socket API is used. */ @@ -1453,11 +1452,10 @@ int Curl_protocol_getsock(struct connectdata *conn, } int Curl_doing_getsock(struct connectdata *conn, - curl_socket_t *socks, - int numsocks) + curl_socket_t *socks) { if(conn && conn->handler->doing_getsock) - return conn->handler->doing_getsock(conn, socks, numsocks); + return conn->handler->doing_getsock(conn, socks); return GETSOCK_BLANK; } |