diff options
| author | Denis Vlasenko | 2008-11-04 21:44:28 +0000 |
|---|---|---|
| committer | Denis Vlasenko | 2008-11-04 21:44:28 +0000 |
| commit | 391ffa19d1e0813c51d51b29117d119b1dc57c49 (patch) | |
| tree | efbd51e4a2371f06a6055c9ecb4b86eca71d9d34 | |
| parent | c51457ca5b0c1938c9b6813c7d72aa7d70fa88f8 (diff) | |
| download | busybox-391ffa19d1e0813c51d51b29117d119b1dc57c49.zip busybox-391ffa19d1e0813c51d51b29117d119b1dc57c49.tar.gz | |
nslookup: tiny shrink (-5 bytes); add comment about res_init
inet_common: cosmetics, no code changes
| -rw-r--r-- | libbb/inet_common.c | 3 | ||||
| -rw-r--r-- | networking/nslookup.c | 17 |
2 files changed, 9 insertions, 11 deletions
diff --git a/libbb/inet_common.c b/libbb/inet_common.c index 3a20b4a..966a021 100644 --- a/libbb/inet_common.c +++ b/libbb/inet_common.c @@ -63,9 +63,6 @@ int FAST_FUNC INET_resolve(const char *name, struct sockaddr_in *s_in, int hostf #ifdef DEBUG res_init(); _res.options |= RES_DEBUG; -#endif - -#ifdef DEBUG bb_error_msg("gethostbyname(%s)", name); #endif hp = gethostbyname(name); diff --git a/networking/nslookup.c b/networking/nslookup.c index 183ae15..73ccb0d 100644 --- a/networking/nslookup.c +++ b/networking/nslookup.c @@ -15,8 +15,11 @@ #include "libbb.h" /* - * I'm only implementing non-interactive mode; - * I totally forgot nslookup even had an interactive mode. + * I'm only implementing non-interactive mode; + * I totally forgot nslookup even had an interactive mode. + * + * This applet is the only user of res_init(). Without it, + * you may avoid pulling in _res global from libc. */ /* Examples of 'standard' nslookup output @@ -51,7 +54,6 @@ static int print_host(const char *hostname, const char *header) { /* We can't use xhost2sockaddr() - we want to get ALL addresses, * not just one */ - struct addrinfo *result = NULL; int rc; struct addrinfo hint; @@ -116,7 +118,7 @@ static void server_print(void) } /* alter the global _res nameserver structure to use - an explicit dns server instead of what is in /etc/resolv.h */ + an explicit dns server instead of what is in /etc/resolv.conf */ static void set_default_dns(char *server) { struct in_addr server_in_addr; @@ -135,18 +137,17 @@ int nslookup_main(int argc, char **argv) * optional DNS server with which to do the lookup. * More than 3 arguments is an error to follow the pattern of the * standard nslookup */ - - if (argc < 2 || *argv[1] == '-' || argc > 3) + if (!argv[1] || argv[1][0] == '-' || argc > 3) bb_show_usage(); /* initialize DNS structure _res used in printing the default * name server and in the explicit name server option feature. */ res_init(); /* rfc2133 says this enables IPv6 lookups */ - /* (but it also says "may be enabled in /etc/resolv.conf|) */ + /* (but it also says "may be enabled in /etc/resolv.conf") */ /*_res.options |= RES_USE_INET6;*/ - if (argc == 3) + if (argv[2]) set_default_dns(argv[2]); server_print(); |