Added check for passing null argument to printf(3) %s directive.

This fixed "directive argument is null" warnings found by gcc
-Wformat-overflow.

diff --git a/sbr/ruserpass.c b/sbr/ruserpass.c
index 77b57a7..4b3d81b 100644 (file)
--- a/sbr/ruserpass.c
+++ b/sbr/ruserpass.c
@@ -153,7 +153,7 @@ ruserpass(const char *host, char **aname, char **apass, int flags)
            if ((pp = getpwuid (getuid())) != NULL)
                myname = pp->pw_name;
        }
            if ((pp = getpwuid (getuid())) != NULL)
                myname = pp->pw_name;
        }

-       printf("Name (%s:%s): ", host, myname);
+       printf("Name (%s:%s): ", host ? host : "", myname);

 
        if (fgets(tmp, sizeof tmp, stdin) == NULL) {
            advise ("tmp", "fgets");
 
        if (fgets(tmp, sizeof tmp, stdin) == NULL) {
            advise ("tmp", "fgets");