[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] qio_dns_resolver_lookup_sync() and SOCKET_ADDRESS_KIND_
From: |
Daniel P. Berrange |
Subject: |
Re: [Qemu-devel] qio_dns_resolver_lookup_sync() and SOCKET_ADDRESS_KIND_FD? |
Date: |
Mon, 3 Apr 2017 12:29:30 +0100 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Thu, Mar 23, 2017 at 05:30:07PM +0100, Markus Armbruster wrote:
> Ignorant question: does qio_dns_resolver_lookup_sync() do the right
> thing for SOCKET_ADDRESS_KIND_FD? The switch doesn't have a case for
> it, and it's unclear whether the default case is really meant for
> SOCKET_ADDRESS_KIND_FD in addition to impossible enumeration values.
No, it is wrong - it should do a 'no op' resolution, not return an
error - ie KIND_FD handling should match KIND_UNIX/KIND_VSOCK.
> Aside: impossible value means your program state is fatally
> compromised. I'd rather assert then.
Sure.
>
>
> int qio_dns_resolver_lookup_sync(QIODNSResolver *resolver,
> SocketAddress *addr,
> size_t *naddrs,
> SocketAddress ***addrs,
> Error **errp)
> {
> switch (addr->type) {
> case SOCKET_ADDRESS_KIND_INET:
> return qio_dns_resolver_lookup_sync_inet(resolver,
> addr,
> naddrs,
> addrs,
> errp);
>
> case SOCKET_ADDRESS_KIND_UNIX:
> case SOCKET_ADDRESS_KIND_VSOCK:
> return qio_dns_resolver_lookup_sync_nop(resolver,
> addr,
> naddrs,
> addrs,
> errp);
>
> default:
> error_setg(errp, "Unknown socket address kind");
> return -1;
> }
> }
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] qio_dns_resolver_lookup_sync() and SOCKET_ADDRESS_KIND_FD?,
Daniel P. Berrange <=