nslookup: tiny shrink (-5 bytes); add comment about res_init

inet_common: cosmetics, no code changes

diff --git a/libbb/inet_common.c b/libbb/inet_common.c
index 3a20b4a..966a021 100644 (file)
--- 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 (file)
--- 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();