MEDIUM: reload: stop passing listener options along with FDs
During a reload operation, we used to send listener options associated
with each passed file descriptor. These were passed as binary contents
for the size of the "options" field in the struct listener. This means
that any flag value change or field size change would be problematic,
the former failing to properly grab certain options, the latter possibly
causing permanent failures during this operation.
Since these two previous commits:
MINOR: reload: determine the foreing binding status from the socket
BUG/MINOR: reload: detect the OS's v6only status before choosing an old socket
we don't need this anymore as the values are determined from the file
descriptor itself.
Let's just turn the previous 32 bits to vestigal space, send them as
zeroes and ignore them on receipt. The only possible side effect is if
someone would want to roll back from a 2.3 to 2.2 or earlier, such options
might be ignored during this reload. But other forthcoming changes might
make this fail as well anyway so that's not a reason for keeping this
behavior.
diff --git a/src/haproxy.c b/src/haproxy.c
index 5b32522..a8d27f8 100644
--- a/src/haproxy.c
+++ b/src/haproxy.c
@@ -1295,34 +1295,30 @@
ha_warning("Inconsistency while transferring sockets\n");
goto out;
}
- memcpy(&xfer_sock->options, &tmpbuf[curoff],
- sizeof(xfer_sock->options));
- curoff += sizeof(xfer_sock->options);
+
+ /* we used to have 32 bits of listener options here but we don't
+ * use them anymore.
+ */
+ curoff += sizeof(int);
/* determine the foreign status directly from the socket itself */
- xfer_sock->options &= ~LI_O_FOREIGN;
if (tcp_is_foreign(fd, xfer_sock->addr.ss_family))
xfer_sock->options |= LI_O_FOREIGN;
/* keep only the v6only flag depending on what's currently
* active on the socket, and always drop the v4v6 one.
*/
+#if defined(IPV6_V6ONLY)
{
int val = 0;
-#if defined(IPV6_V6ONLY)
socklen_t len = sizeof(val);
if (xfer_sock->addr.ss_family == AF_INET6 &&
- getsockopt(fd, IPPROTO_IPV6, IPV6_V6ONLY, &val, &len) != 0)
- val = 0;
-#endif
-
- if (val)
+ getsockopt(fd, IPPROTO_IPV6, IPV6_V6ONLY, &val, &len) == 0 &&
+ val > 0)
xfer_sock->options |= LI_O_V6ONLY;
- else
- xfer_sock->options &= ~LI_O_V6ONLY;
- xfer_sock->options &= ~LI_O_V4V6;
}
+#endif
xfer_sock->fd = fd;
if (xfer_sock_list)