[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] slirp: Split get_dns_addr
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH 2/5] slirp: Split get_dns_addr |
Date: |
Wed, 30 Mar 2016 10:57:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.0 |
On 29.03.2016 01:44, Samuel Thibault wrote:
> Separate get_dns_addr into get_dns_addr_cached and get_dns_addr_resolv_conf
> to make conversion to IPv6 easier.
>
> Signed-off-by: Samuel Thibault <address@hidden>
> ---
> slirp/slirp.c | 53 ++++++++++++++++++++++++++++++++++-------------------
> 1 file changed, 34 insertions(+), 19 deletions(-)
>
> diff --git a/slirp/slirp.c b/slirp/slirp.c
> index 6256c89..2f0d3a3 100644
> --- a/slirp/slirp.c
> +++ b/slirp/slirp.c
> @@ -108,7 +108,28 @@ static void winsock_cleanup(void)
>
> static struct stat dns_addr_stat;
>
> -int get_dns_addr(struct in_addr *pdns_addr)
> +static int get_dns_addr_cached(struct in_addr *pdns_addr)
> +{
> + struct stat old_stat;
> + if ((curtime - dns_addr_time) < TIMEOUT_DEFAULT) {
Cosmetic nit: While you're at it, you could also remove the superfluous
parentheses here.
> + *pdns_addr = dns_addr;
> + return 0;
> + }
> + old_stat = dns_addr_stat;
> + if (stat("/etc/resolv.conf", &dns_addr_stat) != 0) {
> + return -1;
> + }
> + if ((dns_addr_stat.st_dev == old_stat.st_dev)
> + && (dns_addr_stat.st_ino == old_stat.st_ino)
> + && (dns_addr_stat.st_size == old_stat.st_size)
> + && (dns_addr_stat.st_mtime == old_stat.st_mtime)) {
dito.
> + *pdns_addr = dns_addr;
> + return 0;
> + }
> + return 1;
> +}
> +
> +static int get_dns_addr_resolv_conf(struct in_addr *pdns_addr)
> {
> char buff[512];
> char buff2[257];
> @@ -116,24 +137,6 @@ int get_dns_addr(struct in_addr *pdns_addr)
> int found = 0;
> struct in_addr tmp_addr;
>
> - if (dns_addr.s_addr != 0) {
> - struct stat old_stat;
> - if ((curtime - dns_addr_time) < TIMEOUT_DEFAULT) {
> - *pdns_addr = dns_addr;
> - return 0;
> - }
> - old_stat = dns_addr_stat;
> - if (stat("/etc/resolv.conf", &dns_addr_stat) != 0)
> - return -1;
> - if ((dns_addr_stat.st_dev == old_stat.st_dev)
> - && (dns_addr_stat.st_ino == old_stat.st_ino)
> - && (dns_addr_stat.st_size == old_stat.st_size)
> - && (dns_addr_stat.st_mtime == old_stat.st_mtime)) {
> - *pdns_addr = dns_addr;
> - return 0;
> - }
> - }
> -
> f = fopen("/etc/resolv.conf", "r");
> if (!f)
> return -1;
> @@ -173,6 +176,18 @@ int get_dns_addr(struct in_addr *pdns_addr)
> return 0;
> }
>
> +int get_dns_addr(struct in_addr *pdns_addr)
> +{
> + if (dns_addr.s_addr != 0) {
> + int ret;
> + ret = get_dns_addr_cached(pdns_addr);
> + if (ret <= 0) {
> + return ret;
> + }
> + }
> + return get_dns_addr_resolv_conf(pdns_addr);
> +}
> +
> #endif
>
> static void slirp_init_once(void)
>
Reviewed-by: Thomas Huth <address@hidden>
- [Qemu-devel] [PATCH 0/5] ipv4-only and ipv6-only support, Samuel Thibault, 2016/03/28
- [Qemu-devel] [PATCH 5/5] slirp: Add RDNSS advertisement, Samuel Thibault, 2016/03/28
- [Qemu-devel] [PATCH 1/5] slirp: Allow disabling IPv4 or IPv6, Samuel Thibault, 2016/03/28
- [Qemu-devel] [PATCH 2/5] slirp: Split get_dns_addr, Samuel Thibault, 2016/03/28
- Re: [Qemu-devel] [PATCH 2/5] slirp: Split get_dns_addr,
Thomas Huth <=
- [Qemu-devel] [PATCH 3/5] slirp: Add dns6 resolution, Samuel Thibault, 2016/03/28
- [Qemu-devel] [PATCH 4/5] slirp: Support link-local DNS addresses, Samuel Thibault, 2016/03/28