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/tftp.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/tftp.c')
-rw-r--r-- | lib/tftp.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/lib/tftp.c b/lib/tftp.c index c71e5f0aa..2b8084d25 100644 --- a/lib/tftp.c +++ b/lib/tftp.c @@ -157,8 +157,7 @@ static CURLcode tftp_done(struct connectdata *conn, static CURLcode tftp_setup_connection(struct connectdata * conn); static CURLcode tftp_multi_statemach(struct connectdata *conn, bool *done); static CURLcode tftp_doing(struct connectdata *conn, bool *dophase_done); -static int tftp_getsock(struct connectdata *conn, curl_socket_t *socks, - int numsocks); +static int tftp_getsock(struct connectdata *conn, curl_socket_t *socks); static CURLcode tftp_translate_code(tftp_error_t error); @@ -1082,14 +1081,9 @@ static CURLcode tftp_done(struct connectdata *conn, CURLcode status, * The getsock callback * **********************************************************/ -static int tftp_getsock(struct connectdata *conn, curl_socket_t *socks, - int numsocks) +static int tftp_getsock(struct connectdata *conn, curl_socket_t *socks) { - if(!numsocks) - return GETSOCK_BLANK; - socks[0] = conn->sock[FIRSTSOCKET]; - return GETSOCK_READSOCK(0); } |