Skip to content

Commit

Permalink
Fix quicserver binding when duplicate entries exist
Browse files Browse the repository at this point in the history
In testing the quic demos, I found that the quicserver refused to start for me,
indicating an inability to bind a socket to listen on

The problem turned out to be that getaddrinfo on my system was returning
multiple entries, due to the fact that /etc/host maps the localhost host name to
both ipv4 (127.0.0.1) and ipv6 (::1), but returns the latter as an ipv4 mapped
address (specifying family == AF_INET)

It seems like the proper fix would be to modify the /etc/hosts file to not make
that mapping, and indeed that works.  However, since several distribution ship
with this setup, it seems like it is worthwhile to manage it in the server code.

its also that some other application may be bound to a given address/port
leading to failure, which I think could be considered erroneous, as any failure
for the full addrinfo list in quicserver would lead to a complete failure

Fix this by modifying the create_dgram_bio function to count the number of
sockets is successfully binds/listens on, skipping any failures, and only exit
the application if the number of bound sockets is zero.

Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #22559)

(cherry picked from commit fe26b6b)
  • Loading branch information
nhorman authored and mattcaswell committed Nov 2, 2023
1 parent 676f6e2 commit 28932ab
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions util/quicserver.c
Original file line number Diff line number Diff line change
Expand Up @@ -94,23 +94,23 @@ static BIO *create_dgram_bio(int family, const char *hostname, const char *port)
/* Start listening on the socket */
if (!BIO_listen(sock, BIO_ADDRINFO_address(ai), 0)) {
BIO_closesocket(sock);
sock = -1;
continue;
}

/* Set to non-blocking mode */
if (!BIO_socket_nbio(sock, 1)) {
BIO_closesocket(sock);
sock = -1;
continue;
}

break; /* stop searching if we found an addr */
}

/* Free the address information resources we allocated earlier */
BIO_ADDRINFO_free(res);

/* If sock is -1 then we've been unable to connect to the server */
if (sock == -1)
/* If we didn't bind any sockets, fail */
if (ai == NULL)
return NULL;

/* Create a BIO to wrap the socket */
Expand Down

0 comments on commit 28932ab

Please sign in to comment.