aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
Diffstat (limited to 'lib')
-rw-r--r--lib/tftp.c20
1 files changed, 17 insertions, 3 deletions
diff --git a/lib/tftp.c b/lib/tftp.c
index 9af1cf9b5..711bf1720 100644
--- a/lib/tftp.c
+++ b/lib/tftp.c
@@ -559,14 +559,28 @@ CURLcode Curl_tftp_connect(struct connectdata *conn, bool *done)
#ifdef WIN32
/* AF_UNSPEC == 0 (from above calloc) doesn't work on Winsock */
- ((struct sockaddr_in*)&state->local_addr)->sin_family = conn->ip_addr->ai_family;
+
+ /* NOTE: this blatantly assumes IPv4. This should be fixed! */
+ ((struct sockaddr_in*)&state->local_addr)->sin_family =
+ conn->ip_addr->ai_family;
#endif
tftp_set_timeouts(state);
- /* Bind to any interface, random UDP port */
+ /* Bind to any interface, random UDP port.
+ *
+ * We once used the size of the local_addr struct as the third argument for
+ * bind() to better work with IPv6 or whatever size the struct could have,
+ * but we learned that at least Tru64, AIX and IRIX *requires* the size of
+ * that argument to match the exact size of a 'sockaddr_in' struct when
+ * running IPv4-only.
+ *
+ * Therefore we use the size from the address we connected to, which we
+ * assume uses the same IP version and thus hopefully this works for both
+ * IPv4 and IPv6...
+ */
rc = bind(state->sockfd, (struct sockaddr *)&state->local_addr,
- sizeof(state->local_addr));
+ conn->ip_addr->ai_addrlen);
if(rc) {
failf(conn->data, "bind() failed; %s\n",
Curl_strerror(conn,Curl_ourerrno()));