aboutsummaryrefslogtreecommitdiff
path: root/slirp
diff options
context:
space:
mode:
authorEd Swierk <eswierk@aristanetworks.com>2009-08-20 19:00:25 -0700
committerAnthony Liguori <aliguori@us.ibm.com>2009-08-27 20:46:58 -0500
commitce0bd027df9c62766a5417521d0f08f27359d43f (patch)
treed6bdfb3a034bcfef2b98d3a53bb91f01731b2ea8 /slirp
parent382f074371f7dc32a34c944c845b1698e83d8c36 (diff)
downloadqemu-ce0bd027df9c62766a5417521d0f08f27359d43f.zip
qemu-ce0bd027df9c62766a5417521d0f08f27359d43f.tar.gz
qemu-ce0bd027df9c62766a5417521d0f08f27359d43f.tar.bz2
slirp: Remove our_addr code
Three problems with our_addr: - It's determined only once when qemu starts, but the address can change (just like the DNS configuration can). - It's supposed to be the IP address of a host network interface, but there's no guarantee that gethostbyname(gethostname()) actually does that: the host might be a laptop that has only a loopback interface up, or the hostname might be localhost.localdomain, etc. - It's useless at best: get_dns_addr() calls it, there's no reason to send DNS requests to a different IP address if you're running a DNS server on the host and resolv.conf points to 127.0.0.1. These problems are easily solved by removing the code. Signed-off-by: Ed Swierk <eswierk@aristanetworks.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'slirp')
-rw-r--r--slirp/main.h1
-rw-r--r--slirp/slirp.c17
2 files changed, 0 insertions, 18 deletions
diff --git a/slirp/main.h b/slirp/main.h
index 28d92d8..9f22fe1 100644
--- a/slirp/main.h
+++ b/slirp/main.h
@@ -30,7 +30,6 @@ extern char *slirp_tty;
extern char *exec_shell;
extern u_int curtime;
extern fd_set *global_readfds, *global_writefds, *global_xfds;
-extern struct in_addr our_addr;
extern struct in_addr loopback_addr;
extern struct in_addr dns_addr;
extern char *username;
diff --git a/slirp/slirp.c b/slirp/slirp.c
index 827dce7..da04ad1 100644
--- a/slirp/slirp.c
+++ b/slirp/slirp.c
@@ -27,8 +27,6 @@
#include "slirp.h"
#include "hw/hw.h"
-/* host address */
-struct in_addr our_addr;
/* host dns address */
struct in_addr dns_addr;
/* host loopback address */
@@ -117,8 +115,6 @@ static int get_dns_addr(struct in_addr *pdns_addr)
if (sscanf(buff, "nameserver%*[ \t]%256s", buff2) == 1) {
if (!inet_aton(buff2, &tmp_addr))
continue;
- if (tmp_addr.s_addr == loopback_addr.s_addr)
- tmp_addr = our_addr;
/* If it's the first one, set it to dns_addr */
if (!found)
*pdns_addr = tmp_addr;
@@ -149,8 +145,6 @@ static int get_dns_addr(struct in_addr *pdns_addr)
static void slirp_init_once(void)
{
static int initialized;
- struct hostent *he;
- char our_name[256];
#ifdef _WIN32
WSADATA Data;
#endif
@@ -168,17 +162,6 @@ static void slirp_init_once(void)
loopback_addr.s_addr = htonl(INADDR_LOOPBACK);
/* FIXME: This address may change during runtime */
- if (gethostname(our_name, sizeof(our_name)) == 0) {
- he = gethostbyname(our_name);
- if (he) {
- our_addr = *(struct in_addr *)he->h_addr;
- }
- }
- if (our_addr.s_addr == 0) {
- our_addr = loopback_addr;
- }
-
- /* FIXME: This address may change during runtime */
if (get_dns_addr(&dns_addr) < 0) {
dns_addr = loopback_addr;
}