]> git.proxmox.com Git - mirror_qemu.git/commit
util/qemu-sockets: improve ai_flag hints for ipv6 hosts
authorWolfgang Bumiller <w.bumiller@proxmox.com>
Thu, 21 May 2015 12:33:29 +0000 (14:33 +0200)
committerMichael Tokarev <mjt@tls.msk.ru>
Tue, 23 Jun 2015 17:23:39 +0000 (20:23 +0300)
commit3de3d698d942d1116152417f882c897b26b44e41
tree074a9987baf053a03d1ae261142e078166d670c9
parent8684e85ca911b41d6a82ac5bcc5a0bfaba5eb7da
util/qemu-sockets: improve ai_flag hints for ipv6 hosts

*) Do not use AI_ADDRCONFIG on listening sockets, because this flag
makes it impossible to explicitly listen on '127.0.0.1' if no global
ipv4 address is configured additionally, making this a very
uncomfortable option.
*) Add AI_V4MAPPED hint for connecting sockets.

If your system is globally only connected via ipv6 you often still want
to be able to use '127.0.0.1' and 'localhost' (even if localhost doesn't
also have an ipv6 entry).
For example, PVE - unless explicitly asking for insecure mode - uses
ipv4 loopback addresses with QEMU for live migrations tunneled over SSH.
These fail to start because AI_ADDRCONFIG makes getaddrinfo refuse to
work with '127.0.0.1'.

As for the AI_V4MAPPED flag: glibc uses it by default, and providing
non-0 flags removes it. I think it makes sense to use it.

I also want to point out that glibc explicitly sidesteps POSIX standards
when passing 0 as hints by then assuming both AI_V4MAPPED and
AI_ADDRCONFIG (the latter being a rather weird choice IMO), while
according to POSIX.1-2001 it should be assumed 0. (glibc considers its
choice an improvement.)
Since either AI_CANONNAME or AI_PASSIVE are passed in our cases, glibc's
default flags in turn are disabled again unless explicitly added, which
I do with this patch.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
util/qemu-sockets.c