]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: config: do not test an inherited socket again
authorWilly Tarreau <w@1wt.eu>
Wed, 16 Sep 2020 08:43:38 +0000 (10:43 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 16 Sep 2020 20:08:08 +0000 (22:08 +0200)
Since previous patch we know that a successfully bound fd@XXX socket
is returned as its own protocol family from str2sa_range() and not as
AF_CUST_EXISTING_FD anymore o we don't need to check for that case
in str2listener().

src/cfgparse.c

index 59827f45812d72c42f7f0da6a7ae6b13d54462dc..7b5102452decc634d5bcdcae3f73893181d9c303 100644 (file)
@@ -133,25 +133,7 @@ int str2listener(char *str, struct proxy *curproxy, struct bind_conf *bind_conf,
                if (!ss2)
                        goto fail;
 
-               if (ss2->ss_family == AF_CUST_EXISTING_FD) {
-                       socklen_t addr_len;
-
-                       /* We want to attach to an already bound fd whose number
-                        * is in the addr part of ss2 when cast to sockaddr_in.
-                        * Note that by definition there is a single listener.
-                        * We still have to determine the address family to
-                        * register the correct protocol.
-                        */
-                       fd = ((struct sockaddr_in *)ss2)->sin_addr.s_addr;
-                       addr_len = sizeof(*ss2);
-                       if (getsockname(fd, (struct sockaddr *)ss2, &addr_len) == -1) {
-                               memprintf(err, "cannot use file descriptor '%d' : %s.\n", fd, strerror(errno));
-                               goto fail;
-                       }
-
-                       port = end = get_host_port(ss2);
-
-               } else if (ss2->ss_family == AF_CUST_SOCKPAIR) {
+               if (ss2->ss_family == AF_CUST_SOCKPAIR) {
                        socklen_t addr_len;
 
                        fd = ((struct sockaddr_in *)ss2)->sin_addr.s_addr;