qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] [PATCH] Fix regression against commit 4259


From: Jason Wessel
Subject: [Qemu-devel] [PATCH] Fix regression against commit 4259
Date: Sat, 26 Apr 2008 16:30:50 -0500
User-agent: Thunderbird 2.0.0.12 (X11/20080227)

Stuart Brady found a regression based on the 4259 commit where
DNS via slirp to the special address does not work properly.

I verified that DNS to the default special address 10.0.2.3 does not
work.   When using the "special address" for DNS it must be treated as
a special case.  In the original 4259, I did not have a test case for
the loopback DNS.  The translation logic should be used in this case
as well as for any loopback directed requests.

Signed-off-by: Jason Wessel <address@hidden>

---
 slirp/udp.c |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

--- a/slirp/udp.c
+++ b/slirp/udp.c
@@ -318,11 +318,12 @@ int udp_output(struct socket *so, struct
     struct sockaddr_in saddr, daddr;
 
     saddr = *addr;
-    if ((so->so_faddr.s_addr & htonl(0xffffff00)) == special_addr.s_addr &&
-        addr->sin_addr.s_addr == htonl(0x7f000001)) {
-        saddr.sin_addr.s_addr = so->so_faddr.s_addr;
+    if ((so->so_faddr.s_addr & htonl(0xffffff00)) == special_addr.s_addr) {
         if ((so->so_faddr.s_addr & htonl(0x000000ff)) == htonl(0xff))
             saddr.sin_addr.s_addr = alias_addr.s_addr;
+        else if (addr->sin_addr.s_addr == htonl(0x7f000001) ||
+                 ((so->so_faddr.s_addr & htonl(CTL_DNS)) == htonl(CTL_DNS)))
+            saddr.sin_addr.s_addr = so->so_faddr.s_addr;
     }
     daddr.sin_addr = so->so_laddr;
     daddr.sin_port = so->so_lport;




reply via email to

[Prev in Thread] Current Thread [Next in Thread]