Now that I'm seeing the patch in its context, I think it's not the
cleanest way to do it, because we redefine IPV6_TRANSPARENT and
IP_TRANSPARENT when IP_FREEBIND is not defined, while we don't use
this one and the other ones are defined below. Also I'm concerned
about the result of running this with an unsupported kernel, which
will fail the IP_TRANSPARENT syscall and fall back to IP_FREEBIND,
arbitrarily forced to 15 (since it's for Linux) and which I don't
know what it means on FreeBSD.

So I'm thinking again about having a separate USE_FREEBSD_TPROXY
option, I'll see what I can come up with.

Regards,
Willy


Reply via email to