nethack-gwaa-kiwi/nethack
1
0
Fork 0
Code Pull Requests Activity

unix player name handling (trunk only)

Browse Source 
Fix a couple of post-3.4.3 bugs.  MacOSX was unconditionally copying
"player" into plname[] after processing options, clobbering any
name:Somebody value there.  (It took place before command line processing,
so -u Somebody worked ok.)  This removes that, since we're intending to
accomplish the same thing in a different manner.

The revised handling for names "player" and "games" didn't work right when
dash and role were appended to the name in order to try to keep dashes in
usernames intact.  It resulted in first prompting for role selection, then
asking "who are you?" afterwards.

Unfixed bug:  unixmain's appending dash and role to username in order to
preserve usernames with dashes in them doesn't work anymore.  I think the
role/race/gender/alignment stuff introduced way back in 3.3.0 broke it and
apparently no one has noticed....

Miscellaneous:  clean up some complaints from gcc about comparing signed
and unsigned ints.
This commit is contained in:
nethack.rankin
2009-12-13 05:10:42 +00:00
parent 727bd80865
commit 00093a1c8a
1 changed files with 31 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

53
sys/unix/unixmain.c
View File

@@ -1,5 +1,4 @@
/* NetHack 3.5 unixmain.c $Date$ $Revision$ */
/* SCCS Id: @(#)unixmain.c 3.5 2008/01/30 */
/* Copyright (c) Stichting Mathematisch Centrum, Amsterdam, 1985. */
/* NetHack may be freely redistributed. See license for details. */
@@ -194,10 +193,18 @@ char *argv[];
/* wizard mode access is deferred until here */
set_playmode(); /* sets plname to "wizard" for wizard mode */
if (exact_username) {
/*
* FIXME: this no longer works, ever since 3.3.0
* when plnamesuffix() was changed to find
* Name-Role-Race-Gender-Alignment. It removes
* all dashes rather than just the last one,
* regardless of whether whatever follows each
* dash matches role, race, gender, or alignment.
*/
/* guard against user names with hyphens in them */
int len = strlen(plname);
/* append the current role, if any, so that last dash is ours */
if (++len < sizeof plname)
if (++len < (int)sizeof plname)
(void)strncat(strcat(plname, "-"),
pl_character, sizeof plname - len - 1);
}
@@ -466,8 +473,10 @@ boolean wr;
}
#endif /* CHDIR */
/* returns True iff we set plname[] to username which contains a hyphen */
static boolean
whoami() {
whoami()
{
/*
* Who am i? Algorithm: 1. Use name as specified in NETHACKOPTIONS
* 2. Use $USER or $LOGNAME (if 1. fails)
@@ -479,21 +488,19 @@ whoami() {
* Note that we trust the user here; it is possible to play under
* somebody else's name.
*/
#if defined(__APPLE__)
/* Unixisms just confuse the user */
(void) strncpy(plname, "player", sizeof(plname)-1);
#else
register char *s;
if (!*plname) {
register const char *s;
if (*plname) return FALSE;
if(/* !*plname && */ (s = nh_getenv("USER")))
(void) strncpy(plname, s, sizeof(plname)-1);
if(!*plname && (s = nh_getenv("LOGNAME")))
(void) strncpy(plname, s, sizeof(plname)-1);
if(!*plname && (s = getlogin()))
(void) strncpy(plname, s, sizeof(plname)-1);
#endif
return TRUE;
s = nh_getenv("USER");
if (!s || !*s) s = nh_getenv("LOGNAME");
if (!s || !*s) s = getlogin();
if (s && *s) {
(void) strncpy(plname, s, sizeof plname - 1);
if (index(plname, '-')) return TRUE;
}
}
return FALSE;
}
void
@@ -619,23 +626,25 @@ char *optstr;
}
static struct passwd *
get_unix_pw(){
get_unix_pw()
{
char *user;
int uid;
unsigned uid;
static struct passwd *pw = (struct passwd *)0;
if(pw) return pw; /* cache answer */
uid = getuid();
uid = (unsigned)getuid();
user = getlogin();
if (user) {
pw = getpwnam(user);
if (pw && (pw->pw_uid != uid)) pw = 0;
if (pw && ((unsigned)pw->pw_uid != uid)) pw = 0;
}
if (pw == 0) {
user = nh_getenv("USER");
if (user) {
pw = getpwnam(user);
if (pw && (pw->pw_uid != uid)) pw = 0;
if (pw && ((unsigned)pw->pw_uid != uid)) pw = 0;
}
if (pw == 0) {
pw = getpwuid(uid);