[PATCH] slirp: Allow non-local DNS address when restrict is off

2019-10-01 Thread Samuel Thibault 
This can be used to set a DNS server to be used by the guest which is
different from the one configured on the host.

Buglink: https://bugs.launchpad.net/qemu/+bug/1010484
Signed-off-by: Samuel Thibault 
---
 net/slirp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/slirp.c b/net/slirp.c
index f42f496641..c4334ee876 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -456,7 +456,7 @@ static int net_slirp_init(NetClientState *peer, const char 
*model,
 error_setg(errp, "Failed to parse DNS");
 return -1;
 }
-if ((dns.s_addr & mask.s_addr) != net.s_addr) {
+if (restricted && (dns.s_addr & mask.s_addr) != net.s_addr) {
 error_setg(errp, "DNS doesn't belong to network");
 return -1;
 }
@@ -522,7 +522,7 @@ static int net_slirp_init(NetClientState *peer, const char 
*model,
 error_setg(errp, "Failed to parse IPv6 DNS");
 return -1;
 }
-if (!in6_equal_net(_prefix, _dns, vprefix6_len)) {
+if (restricted && !in6_equal_net(_prefix, _dns, vprefix6_len)) 
{
 error_setg(errp, "IPv6 DNS doesn't belong to network");
 return -1;
 }
-- 
2.23.0

Re: [PATCH] slirp: Allow non-local DNS address when restrict is off

2019-10-01 Thread Thomas Huth 
On 01/10/2019 11.04, Philippe Mathieu-Daudé wrote:
> Hi Samuel,
> 
> On 9/29/19 8:13 PM, Samuel Thibault wrote:
>> This can be used to set a DNS server to be used by the guest which is
>> different from the one configured on the host.
>>
>> This fixes LP 1010484.
> 
> Wow, 7 years old...
> 
> Can you use this format, easier to understand for newcomers:
> 
> Fixes: https://bugs.launchpad.net/qemu/+bug/1010484

That should be "Buglink:" instead. "Fixes:" is for referencing previous
commits. See our submit-a-patch page for more information:
https://wiki.qemu.org/Contribute/SubmitAPatch#Write_a_meaningful_commit_message

 Thomas

Re: [PATCH] slirp: Allow non-local DNS address when restrict is off

2019-10-01 Thread Philippe Mathieu-Daudé 

Hi Samuel,

On 9/29/19 8:13 PM, Samuel Thibault wrote:

This can be used to set a DNS server to be used by the guest which is
different from the one configured on the host.

This fixes LP 1010484.


Wow, 7 years old...

Can you use this format, easier to understand for newcomers:

Fixes: https://bugs.launchpad.net/qemu/+bug/1010484



Signed-off-by: Samuel Thibault 
---
  net/slirp.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/slirp.c b/net/slirp.c
index f42f496641..4d158b0542 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -456,7 +456,7 @@ static int net_slirp_init(NetClientState *peer, const char 
*model,
  error_setg(errp, "Failed to parse DNS");
  return -1;
  }
-if ((dns.s_addr & mask.s_addr) != net.s_addr) {
+if (restricted && (dns.s_addr & mask.s_addr) != net.s_addr) {
  error_setg(errp, "DNS doesn't belong to network");
  return -1;
  }



What about the IPv6 DNS?

Regards,

Phil.

Re: [PATCH] slirp: Allow non-local DNS address when restrict is off

2019-09-29 Thread Marc-André Lureau 
On Sun, Sep 29, 2019 at 10:13 PM Samuel Thibault
 wrote:
>
> This can be used to set a DNS server to be used by the guest which is
> different from the one configured on the host.
>
> This fixes LP 1010484.
>
> Signed-off-by: Samuel Thibault 

Reviewed-by: Marc-André Lureau 

> ---
>  net/slirp.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/slirp.c b/net/slirp.c
> index f42f496641..4d158b0542 100644
> --- a/net/slirp.c
> +++ b/net/slirp.c
> @@ -456,7 +456,7 @@ static int net_slirp_init(NetClientState *peer, const 
> char *model,
>  error_setg(errp, "Failed to parse DNS");
>  return -1;
>  }
> -if ((dns.s_addr & mask.s_addr) != net.s_addr) {
> +if (restricted && (dns.s_addr & mask.s_addr) != net.s_addr) {
>  error_setg(errp, "DNS doesn't belong to network");
>  return -1;
>  }
> --
> 2.23.0
>
>


-- 
Marc-André Lureau

[PATCH] slirp: Allow non-local DNS address when restrict is off

2019-09-29 Thread Samuel Thibault 
This can be used to set a DNS server to be used by the guest which is
different from the one configured on the host.

This fixes LP 1010484.

Signed-off-by: Samuel Thibault 
---
 net/slirp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/slirp.c b/net/slirp.c
index f42f496641..4d158b0542 100644
--- a/net/slirp.c
+++ b/net/slirp.c
@@ -456,7 +456,7 @@ static int net_slirp_init(NetClientState *peer, const char 
*model,
 error_setg(errp, "Failed to parse DNS");
 return -1;
 }
-if ((dns.s_addr & mask.s_addr) != net.s_addr) {
+if (restricted && (dns.s_addr & mask.s_addr) != net.s_addr) {
 error_setg(errp, "DNS doesn't belong to network");
 return -1;
 }
-- 
2.23.0