On Tue, Jan 18, 2022 at 11:19:18AM -0600, Eric Blake wrote:
On Tue, Jan 18, 2022 at 02:48:33PM +0100, Laszlo Ersek wrote:
> + case '4':
> + tcpip_sock_af = AF_INET;
> + break;
> +
> + case '6':
> + tcpip_sock_af = AF_INET6;
> + break;
Thus, if the user uses nbdkit -46 (or -64), the last one specified
silently overrides the earlier one, rather than being diagnosed as
conflicting or explicitly permitting both. The override effect
matches the long-option naming --ipv4-only, so I can live with it, but
I'm also open to the idea of explicitly adding code to diagnose both
options at the same time as an error, if we think that's friendlier
than silent override.
Is that actually an error? -4 -6 says you want to listen on IPv4 and
IPv6 which is redundant but not wrong.
Rich.
--
Richard Jones, Virtualization Group, Red Hat
http://people.redhat.com/~rjones
Read my programming and virtualization blog:
http://rwmj.wordpress.com
virt-p2v converts physical machines to virtual machines. Boot with a
live CD or over the network (PXE) and turn machines into KVM guests.
http://libguestfs.org/virt-v2v