register user should not default to "" it leaves too many

chances to screw things up, by just hitting return.

diff --git a/clients/moira/user.c b/clients/moira/user.c
index ff24c17664e6d80036e035d92ccee6143a93180e..da9514d9e9ef65cd8515e48ad70ba8a30fee5045 100644 (file)
--- a/clients/moira/user.c
+++ b/clients/moira/user.c
@@ -630,11 +630,16 @@ RegisterUser()
        return(DM_NORMAL);
     }
 
-    if ( ((login = args[1] = GetLoginName()) == NULL) ||
+    sprintf(temp_buf, "u%s", args[0]);
+    login = strsave(temp_buf);
+    if ( (GetValueFromUser("Login name for this user? ", &login) == SUB_ERROR) ||
        ( GetFSTypes(&fstype, FALSE) == SUB_ERROR ) ) {
+        args[1] = login;
        FreeInfo(args);    /* This work because the NULL temination is ok. */
        return(DM_NORMAL);
     }
+    Put_message("KERBEROS code not added, did not reserve name with kerberos.");
+    args[1] = login;
     args[2] = fstype;
     args[3] = NULL;