From 391ffa19d1e0813c51d51b29117d119b1dc57c49 Mon Sep 17 00:00:00 2001
From: Denis Vlasenko <vda.linux@googlemail.com>
Date: Tue, 4 Nov 2008 21:44:28 +0000
Subject: [PATCH] nslookup: tiny shrink (-5 bytes); add comment about res_init
 inet_common: cosmetics, no code changes

---
 libbb/inet_common.c   |  3 ---
 networking/nslookup.c | 17 +++++++++--------
 2 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/libbb/inet_common.c b/libbb/inet_common.c
index 3a20b4a16..966a021e5 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 183ae152d..73ccb0ddd 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();