Adapt BIO_new_accept() to call BIO_set_accept_name()
authorjamercee <jim@carroll.com>
Wed, 3 Aug 2016 15:31:46 +0000 (11:31 -0400)
committerRich Salz <rsalz@openssl.org>
Wed, 10 Aug 2016 15:07:42 +0000 (11:07 -0400)
Commit 417be66 broken BIO_new_accept() by changing the definition of the
macro BIO_set_accept_port() which stopped acpt_ctrl() from calling
BIO_parse_hostserv(). This commit completes the series of changes
initiated in 417be66.

Updated pods to reflect new definition introduced by 417be66.

Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/1386)

crypto/bio/bss_acpt.c
doc/crypto/BIO_s_accept.pod

index 628da9e6348b53282e3b605e89e369e2875a9995..6fb971a50d513ecfd7120e5317a5f724868d7b7b 100644 (file)
@@ -548,7 +548,7 @@ BIO *BIO_new_accept(const char *str)
     ret = BIO_new(BIO_s_accept());
     if (ret == NULL)
         return (NULL);
-    if (BIO_set_accept_port(ret, str))
+    if (BIO_set_accept_name(ret, str))
         return (ret);
     BIO_free(ret);
     return (NULL);
index 724f1480df99620a390588cae6cbec3cfa267ed3..174bd73ce150061201c34fa449994031c8d53adb 100644 (file)
@@ -2,9 +2,9 @@
 
 =head1 NAME
 
-BIO_s_accept, BIO_set_accept_port, BIO_get_accept_port, BIO_new_accept,
-BIO_set_nbio_accept, BIO_set_accept_bios, BIO_set_bind_mode,
-BIO_get_bind_mode, BIO_do_accept - accept BIO
+BIO_s_accept, BIO_set_accept_name, BIO_get_name, BIO_set_accept_port,
+BIO_get_accept_port, BIO_new_accept, BIO_set_nbio_accept, BIO_set_accept_bios,
+BIO_set_bind_mode, BIO_get_bind_mode, BIO_do_accept - accept BIO
 
 =head1 SYNOPSIS
 
@@ -12,6 +12,9 @@ BIO_get_bind_mode, BIO_do_accept - accept BIO
 
  const BIO_METHOD *BIO_s_accept(void);
 
+ long BIO_set_accept_name(BIO *b, char *name);
+ char *BIO_get_accept_name(BIO *b);
+
  long BIO_set_accept_port(BIO *b, char *name);
  char *BIO_get_accept_port(BIO *b);
 
@@ -52,17 +55,17 @@ incoming connection.
 BIO_get_fd() and BIO_set_fd() can be called to retrieve or set
 the accept socket. See L<BIO_s_fd(3)>
 
-BIO_set_accept_port() uses the string B<name> to set the accept
-port. The port is represented as a string of the form "host:port",
+BIO_set_accept_name() uses the string B<name> to set the accept
+name. The name is represented as a string of the form "host:port",
 where "host" is the interface to use and "port" is the port.
 The host can be "*" or empty which is interpreted as meaning
 any interface.  If the host is an IPv6 address, it has to be
 enclosed in brackets, for example "[::1]:https".  "port" has the
-same syntax as the port specified in BIO_set_conn_port() for
+same syntax as the port specified in BIO_set_conn_name() for
 connect BIOs, that is it can be a numerical port string or a
 string to lookup using getservbyname() and a string table.
 
-BIO_new_accept() combines BIO_new() and BIO_set_accept_port() into
+BIO_new_accept() combines BIO_new() and BIO_set_accept_name() into
 a single call: that is it creates a new accept BIO with port
 B<host_port>.
 
@@ -134,17 +137,19 @@ then it is an indication that an accept attempt would block: the application
 should take appropriate action to wait until the underlying socket has
 accepted a connection and retry the call.
 
-BIO_set_accept_port(), BIO_get_accept_port(), BIO_set_nbio_accept(),
-BIO_set_accept_bios(), BIO_set_bind_mode(), BIO_get_bind_mode() and
-BIO_do_accept() are macros.
+BIO_set_accept_name(), BIO_get_accept_name(), BIO_set_accept_port(),
+BIO_get_accept_port(), BIO_set_nbio_accept(), BIO_set_accept_bios(),
+BIO_set_bind_mode(), BIO_get_bind_mode() and BIO_do_accept() are macros.
 
 =head1 RETURN VALUES
 
 BIO_do_accept(),
-BIO_set_accept_port(), BIO_set_nbio_accept(), BIO_set_accept_bios(),
-and BIO_set_bind_mode(), return 1 for success and 0 or -1 for failure.
+BIO_set_accept_name(), BIO_set_accept_port(), BIO_set_nbio_accept(),
+BIO_set_accept_bios(), and BIO_set_bind_mode(), return 1 for success and 0 or
+-1 for failure.
 
-BIO_get_accept_port() returns the port name or NULL on error.
+BIO_set_accept_name() and BIO_get_accept_port() returns the port name or NULL
+on error.
 
 BIO_get_bind_mode() returns the set of B<BIO_BIND> flags, or -1 on failure.