diff options
| author | Daniel Stenberg <daniel@haxx.se> | 2019-07-30 11:02:03 +0200 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2019-07-30 23:16:44 +0200 |
| commit | a55faf33d4b868c8e8173e7aabfe1e286a1a7658 (patch) | |
| tree | 415811788599dee37eed1e52b7c7595e794e3315 /lib/tftp.c | |
| parent | cb542ac4d085e9446a9dfc6632fad9a851b507fe (diff) | |
| download | curl-a55faf33d4b868c8e8173e7aabfe1e286a1a7658.tar.gz | |
cleanup: remove the 'numsocks' argument used in many places
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.
Closes #4169
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); } |