]> code.citadel.org Git - citadel.git/blobdiff - citadel/user_ops.c
* Variable names, comments, documentation, etc... removed the acronym 'BBS'
[citadel.git] / citadel / user_ops.c
index b657187f67288a147da22e7bd663b0aa11522078..f375637e16d30342a25121a436faccf51717715c 100644 (file)
@@ -33,7 +33,6 @@
 #endif
 
 #include <string.h>
-#include <syslog.h>
 #include <limits.h>
 #ifndef ENABLE_CHKPWD
 #include "auth.h"
 #include "citserver.h"
 
 
+/*
+ * makeuserkey() - convert a username into the format used as a database key
+ *                 (it's just the username converted into lower case)
+ */
+static INLINE void makeuserkey(char *key, char *username) {
+       int i, len;
+
+       len = strlen(username);
+       for (i=0; i<=len; ++i) {
+               key[i] = tolower(username[i]);
+       }
+}
+
+
 /*
  * getuser()  -  retrieve named user into supplied buffer.
  *               returns 0 on success
  */
-int getuser(struct user *usbuf, char name[])
+int getuser(struct ctdluser *usbuf, char name[])
 {
 
-       char lowercase_name[USERNAME_SIZE];
+       char usernamekey[USERNAME_SIZE];
        char sysuser_name[USERNAME_SIZE];
-       int a;
        struct cdbdata *cdbus;
        int using_sysuser = 0;
 
-       memset(usbuf, 0, sizeof(struct user));
+       if (usbuf != NULL) {
+               memset(usbuf, 0, sizeof(struct ctdluser));
+       }
 
 #ifdef ENABLE_AUTOLOGIN
        if (CtdlAssociateSystemUser(sysuser_name, name) == 0) {
@@ -76,25 +90,21 @@ int getuser(struct user *usbuf, char name[])
 #endif
 
        if (using_sysuser) {
-               for (a = 0; a <= strlen(sysuser_name); ++a) {
-                       lowercase_name[a] = tolower(sysuser_name[a]);
-               }
+               makeuserkey(usernamekey, sysuser_name);
        }
        else {
-               for (a = 0; a <= strlen(name); ++a) {
-                       if (a < sizeof(lowercase_name))
-                               lowercase_name[a] = tolower(name[a]);
-               }
+               makeuserkey(usernamekey, name);
        }
-       lowercase_name[sizeof(lowercase_name) - 1] = 0;
 
-       cdbus = cdb_fetch(CDB_USERSUPP, lowercase_name, strlen(lowercase_name));
+       cdbus = cdb_fetch(CDB_USERS, usernamekey, strlen(usernamekey));
        if (cdbus == NULL) {    /* user not found */
                return(1);
        }
-       memcpy(usbuf, cdbus->ptr,
-              ((cdbus->len > sizeof(struct user)) ?
-               sizeof(struct user) : cdbus->len));
+       if (usbuf != NULL) {
+               memcpy(usbuf, cdbus->ptr,
+                       ((cdbus->len > sizeof(struct ctdluser)) ?
+                        sizeof(struct ctdluser) : cdbus->len));
+       }
        cdb_free(cdbus);
 
        return (0);
@@ -104,13 +114,13 @@ int getuser(struct user *usbuf, char name[])
 /*
  * lgetuser()  -  same as getuser() but locks the record
  */
-int lgetuser(struct user *usbuf, char *name)
+int lgetuser(struct ctdluser *usbuf, char *name)
 {
        int retcode;
 
        retcode = getuser(usbuf, name);
        if (retcode == 0) {
-               begin_critical_section(S_USERSUPP);
+               begin_critical_section(S_USERS);
        }
        return (retcode);
 }
@@ -119,21 +129,16 @@ int lgetuser(struct user *usbuf, char *name)
 /*
  * putuser()  -  write user buffer into the correct place on disk
  */
-void putuser(struct user *usbuf)
+void putuser(struct ctdluser *usbuf)
 {
-       char lowercase_name[USERNAME_SIZE];
-       int a;
+       char usernamekey[USERNAME_SIZE];
 
-       for (a = 0; a <= strlen(usbuf->fullname); ++a) {
-               if (a < sizeof(lowercase_name))
-                       lowercase_name[a] = tolower(usbuf->fullname[a]);
-       }
-       lowercase_name[sizeof(lowercase_name) - 1] = 0;
+       makeuserkey(usernamekey, usbuf->fullname);
 
        usbuf->version = REV_LEVEL;
-       cdb_store(CDB_USERSUPP,
-                 lowercase_name, strlen(lowercase_name),
-                 usbuf, sizeof(struct user));
+       cdb_store(CDB_USERS,
+                 usernamekey, strlen(usernamekey),
+                 usbuf, sizeof(struct ctdluser));
 
 }
 
@@ -141,10 +146,10 @@ void putuser(struct user *usbuf)
 /*
  * lputuser()  -  same as putuser() but locks the record
  */
-void lputuser(struct user *usbuf)
+void lputuser(struct ctdluser *usbuf)
 {
        putuser(usbuf);
-       end_critical_section(S_USERSUPP);
+       end_critical_section(S_USERS);
 }
 
 /*
@@ -199,8 +204,8 @@ void put_visit(struct visit *newvisit)
  * Define a relationship between a user and a room
  */
 void CtdlSetRelationship(struct visit *newvisit,
-                        struct user *rel_user,
-                        struct room *rel_room)
+                        struct ctdluser *rel_user,
+                        struct ctdlroom *rel_room)
 {
 
 
@@ -218,8 +223,8 @@ void CtdlSetRelationship(struct visit *newvisit,
  * Locate a relationship between a user and a room
  */
 void CtdlGetRelationship(struct visit *vbuf,
-                        struct user *rel_user,
-                        struct room *rel_room)
+                        struct ctdluser *rel_user,
+                        struct ctdlroom *rel_room)
 {
 
        char IndexBuf[32];
@@ -256,7 +261,7 @@ void CtdlGetRelationship(struct visit *vbuf,
 }
 
 
-void MailboxName(char *buf, size_t n, const struct user *who, const char *prefix)
+void MailboxName(char *buf, size_t n, const struct ctdluser *who, const char *prefix)
 {
        snprintf(buf, n, "%010ld.%s", who->usernum, prefix);
 }
@@ -299,20 +304,20 @@ int is_room_aide(void)
  * WARNING: don't use this function unless you absolutely have to.  It does
  *          a sequential search and therefore is computationally expensive.
  */
-int getuserbynumber(struct user *usbuf, long int number)
+int getuserbynumber(struct ctdluser *usbuf, long int number)
 {
        struct cdbdata *cdbus;
 
-       cdb_rewind(CDB_USERSUPP);
+       cdb_rewind(CDB_USERS);
 
-       while (cdbus = cdb_next_item(CDB_USERSUPP), cdbus != NULL) {
-               memset(usbuf, 0, sizeof(struct user));
+       while (cdbus = cdb_next_item(CDB_USERS), cdbus != NULL) {
+               memset(usbuf, 0, sizeof(struct ctdluser));
                memcpy(usbuf, cdbus->ptr,
-                      ((cdbus->len > sizeof(struct user)) ?
-                       sizeof(struct user) : cdbus->len));
+                      ((cdbus->len > sizeof(struct ctdluser)) ?
+                       sizeof(struct ctdluser) : cdbus->len));
                cdb_free(cdbus);
                if (usbuf->usernum == number) {
-                       cdb_close_cursor(CDB_USERSUPP);
+                       cdb_close_cursor(CDB_USERS);
                        return (0);
                }
        }
@@ -350,22 +355,55 @@ int CtdlLoginExistingUser(char *trythisname)
 {
        char username[SIZ];
        int found_user;
+       struct recptypes *valid = NULL;
 
        if (trythisname == NULL) return login_not_found;
-       safestrncpy(username, trythisname, sizeof username);
-       strproc(username);
+       safestrncpy(username, trythisname, USERNAME_SIZE);
+       striplt(username);
 
        if ((CC->logged_in)) {
                return login_already_logged_in;
        }
 
+       /* First, try to log in as if the supplied name is a display name */
        found_user = getuser(&CC->user, username);
 
+       /* If that didn't work, try to log in as if the supplied name
+        * is an e-mail address
+        */
+       if (found_user != 0) {
+               valid = validate_recipients(username);
+               if (valid != NULL) {
+                       if (valid->num_local == 1) {
+                               found_user = getuser(&CC->user,
+                                               valid->recp_local);
+                       }
+                       free(valid);
+               }
+       }
+
+#ifdef ENABLE_AUTOLOGIN
+       /* If we haven't found the account yet, and the supplied name
+        * is a login name on the underlying host system, create the
+        * account.
+        */
+       if (found_user != 0) {
+               struct passwd *p = (struct passwd *) getpwnam(username);
+
+               if (p != NULL) {
+                       create_user(username, 0);
+                       found_user = getuser(&CC->user, username);
+               }
+       }
+#endif /* ENABLE_AUTOLOGIN */
+
+       /* Did we find something? */
        if (found_user == 0) {
                if (((CC->nologin)) && (CC->user.axlevel < 6)) {
                        return login_too_many_users;
                } else {
-                       strcpy(CC->curr_user, CC->user.fullname);
+                       safestrncpy(CC->curr_user, CC->user.fullname,
+                                       sizeof CC->curr_user);
                        return login_ok;
                }
        }
@@ -379,17 +417,16 @@ int CtdlLoginExistingUser(char *trythisname)
  */
 void cmd_user(char *cmdbuf)
 {
-       char username[SIZ];
+       char username[256];
        int a;
 
-       extract(username, cmdbuf, 0);
-       username[25] = 0;
-       strproc(username);
+       extract_token(username, cmdbuf, 0, '|', sizeof username);
+       striplt(username);
 
        a = CtdlLoginExistingUser(username);
        switch (a) {
        case login_already_logged_in:
-               cprintf("%d Already logged in.\n", ERROR);
+               cprintf("%d Already logged in.\n", ERROR + ALREADY_LOGGED_IN);
                return;
        case login_too_many_users:
                cprintf("%d %s: "
@@ -403,9 +440,10 @@ void cmd_user(char *cmdbuf)
                        MORE_DATA, CC->curr_user);
                return;
        case login_not_found:
-               cprintf("%d %s not found.\n", ERROR, username);
+               cprintf("%d %s not found.\n", ERROR + NO_SUCH_USER, username);
                return;
-               cprintf("%d Internal error\n", ERROR);
+       default:
+               cprintf("%d Internal error\n", ERROR + INTERNAL_ERROR);
        }
 }
 
@@ -418,8 +456,7 @@ void session_startup(void)
 {
        int i;
 
-       syslog(LOG_NOTICE, "session %d: user <%s> logged in",
-              CC->cs_pid, CC->curr_user);
+       lprintf(CTDL_NOTICE, "<%s> logged in\n", CC->curr_user);
 
        lgetuser(&CC->user, CC->curr_user);
        ++(CC->user.timescalled);
@@ -450,8 +487,8 @@ void session_startup(void)
        /* Create any personal rooms required by the system.
         * (Technically, MAILROOM should be there already, but just in case...)
         */
-       create_room(MAILROOM, 4, "", 0, 1, 0);
-       create_room(SENTITEMS, 4, "", 0, 1, 0);
+       create_room(MAILROOM, 4, "", 0, 1, 0, VIEW_MAILBOX);
+       create_room(SENTITEMS, 4, "", 0, 1, 0, VIEW_MAILBOX);
 
        /* Run any startup routines registered by loadable modules */
        PerformSessionHooks(EVT_LOGIN);
@@ -477,6 +514,16 @@ void logged_in_response(void)
  */
 void logout(struct CitContext *who)
 {
+       /*
+        * Clear out some session data.  Most likely, the CitContext for this
+        * session is about to get nuked when the session disconnects, but
+        * since it's possible to log in again without reconnecting, we cannot
+        * make that assumption.
+        */
+       strcpy(who->fake_username, "");
+       strcpy(who->fake_postname, "");
+       strcpy(who->fake_hostname, "");
+       strcpy(who->fake_roomname, "");
        who->logged_in = 0;
 
        /*
@@ -503,6 +550,11 @@ void logout(struct CitContext *who)
 
        /* Do modular stuff... */
        PerformSessionHooks(EVT_LOGOUT);
+
+       /* Free any output buffers */
+       if (who->output_buffer != NULL) {
+               unbuffer_output();
+       }
 }
 
 #ifdef ENABLE_CHKPWD
@@ -517,13 +569,13 @@ static int validpw(uid_t uid, const char *pass)
        char buf[24];
 
        if (pipe(pipev)) {
-               lprintf(1, "pipe failed (%s): denying autologin access for "
+               lprintf(CTDL_ERR, "pipe failed (%s): denying autologin access for "
                        "uid %ld\n", strerror(errno), (long)uid);
                return 0;
        }
        switch (pid = fork()) {
        case -1:
-               lprintf(1, "fork failed (%s): denying autologin access for "
+               lprintf(CTDL_ERR, "fork failed (%s): denying autologin access for "
                        "uid %ld\n", strerror(errno), (long)uid);
                close(pipev[0]);
                close(pipev[1]);
@@ -537,8 +589,8 @@ static int validpw(uid_t uid, const char *pass)
                }
                close(pipev[0]);
 
-               execl(BBSDIR "/chkpwd", BBSDIR "/chkpwd", NULL);
-               perror(BBSDIR "/chkpwd");
+               execl(CTDLDIR "/chkpwd", CTDLDIR "/chkpwd", NULL);
+               perror(CTDLDIR "/chkpwd");
                exit(1);
        }
 
@@ -551,7 +603,7 @@ static int validpw(uid_t uid, const char *pass)
 
        while (waitpid(pid, &status, 0) == -1)
                if (errno != EINTR) {
-                       lprintf(1, "waitpid failed (%s): denying autologin "
+                       lprintf(CTDL_ERR, "waitpid failed (%s): denying autologin "
                                "access for uid %ld\n",
                                strerror(errno), (long)uid);
                        return 0;
@@ -575,29 +627,29 @@ int CtdlTryPassword(char *password)
        int code;
 
        if ((CC->logged_in)) {
-               lprintf(5, "CtdlTryPassword: already logged in\n");
+               lprintf(CTDL_WARNING, "CtdlTryPassword: already logged in\n");
                return pass_already_logged_in;
        }
        if (!strcmp(CC->curr_user, NLI)) {
-               lprintf(5, "CtdlTryPassword: no user selected\n");
+               lprintf(CTDL_WARNING, "CtdlTryPassword: no user selected\n");
                return pass_no_user;
        }
        if (getuser(&CC->user, CC->curr_user)) {
-               lprintf(5, "CtdlTryPassword: internal error\n");
+               lprintf(CTDL_ERR, "CtdlTryPassword: internal error\n");
                return pass_internal_error;
        }
        if (password == NULL) {
-               lprintf(5, "CtdlTryPassword: NULL password string supplied\n");
+               lprintf(CTDL_INFO, "CtdlTryPassword: NULL password string supplied\n");
                return pass_wrong_password;
        }
        code = (-1);
 
 
 #ifdef ENABLE_AUTOLOGIN
-       /* A uid of BBSUID or -1 indicates that this user exists only in
+       /* A uid of CTDLUID or -1 indicates that this user exists only in
         * Citadel, not in the underlying operating system.
         */
-       if ( (CC->user.uid == BBSUID) || (CC->user.uid == (-1)) ) {
+       if ( (CC->user.uid == CTDLUID) || (CC->user.uid == (-1)) ) {
                strproc(password);
                strproc(CC->user.password);
                code = strcasecmp(CC->user.password, password);
@@ -624,7 +676,7 @@ int CtdlTryPassword(char *password)
                do_login();
                return pass_ok;
        } else {
-               lprintf(3, "Bad password specified for <%s>\n", CC->curr_user);
+               lprintf(CTDL_WARNING, "Bad password specified for <%s>\n", CC->curr_user);
                return pass_wrong_password;
        }
 }
@@ -632,28 +684,26 @@ int CtdlTryPassword(char *password)
 
 void cmd_pass(char *buf)
 {
-       char password[SIZ];
+       char password[256];
        int a;
 
-       extract(password, buf, 0);
+       extract_token(password, buf, 0, '|', sizeof password);
        a = CtdlTryPassword(password);
 
        switch (a) {
        case pass_already_logged_in:
-               cprintf("%d Already logged in.\n", ERROR);
+               cprintf("%d Already logged in.\n", ERROR + ALREADY_LOGGED_IN);
                return;
        case pass_no_user:
                cprintf("%d You must send a name with USER first.\n",
-                       ERROR);
+                       ERROR + USERNAME_REQUIRED);
                return;
        case pass_wrong_password:
-               cprintf("%d Wrong password.\n", ERROR);
+               cprintf("%d Wrong password.\n", ERROR + PASSWORD_REQUIRED);
                return;
        case pass_ok:
                logged_in_response();
                return;
-               cprintf("%d Can't find user record!\n",
-                       ERROR + INTERNAL_ERROR);
        }
 }
 
@@ -665,18 +715,15 @@ void cmd_pass(char *buf)
 int purge_user(char pname[])
 {
        char filename[64];
-       struct user usbuf;
-       char lowercase_name[USERNAME_SIZE];
-       int a;
+       struct ctdluser usbuf;
+       char usernamekey[USERNAME_SIZE];
        struct CitContext *ccptr;
        int user_is_logged_in = 0;
 
-       for (a = 0; a <= strlen(pname); ++a) {
-               lowercase_name[a] = tolower(pname[a]);
-       }
+       makeuserkey(usernamekey, pname);
 
        if (getuser(&usbuf, pname) != 0) {
-               lprintf(5, "Cannot purge user <%s> - not found\n", pname);
+               lprintf(CTDL_ERR, "Cannot purge user <%s> - not found\n", pname);
                return (ERROR + NO_SUCH_USER);
        }
        /* Don't delete a user who is currently logged in.  Instead, just
@@ -692,21 +739,21 @@ int purge_user(char pname[])
        }
        end_critical_section(S_SESSION_TABLE);
        if (user_is_logged_in == 1) {
-               lprintf(5, "User <%s> is logged in; not deleting.\n", pname);
+               lprintf(CTDL_WARNING, "User <%s> is logged in; not deleting.\n", pname);
                usbuf.axlevel = 0;
                putuser(&usbuf);
                return (1);
        }
-       lprintf(5, "Deleting user <%s>\n", pname);
+       lprintf(CTDL_NOTICE, "Deleting user <%s>\n", pname);
 
        /* Perform any purge functions registered by server extensions */
-       PerformUserHooks(usbuf.fullname, usbuf.usernum, EVT_PURGEUSER);
+       PerformUserHooks(&usbuf, EVT_PURGEUSER);
 
        /* delete any existing user/room relationships */
        cdb_delete(CDB_VISIT, &usbuf.usernum, sizeof(long));
 
        /* delete the userlog entry */
-       cdb_delete(CDB_USERSUPP, lowercase_name, strlen(lowercase_name));
+       cdb_delete(CDB_USERS, usernamekey, strlen(usernamekey));
 
        /* remove the user's bio file */
        snprintf(filename, sizeof filename, "./bio/%ld", usbuf.usernum);
@@ -729,23 +776,25 @@ int purge_user(char pname[])
  */
 int create_user(char *newusername, int become_user)
 {
-       struct user usbuf;
-       struct room qrbuf;
-       struct passwd *p = NULL;
-       char username[SIZ];
+       struct ctdluser usbuf;
+       struct ctdlroom qrbuf;
+       char username[256];
        char mailboxname[ROOMNAMELEN];
        uid_t uid;
 
-       strcpy(username, newusername);
+       safestrncpy(username, newusername, sizeof username);
        strproc(username);
 
 #ifdef ENABLE_AUTOLOGIN
-       p = (struct passwd *) getpwnam(username);
-       if (p != NULL) {
-               extract_token(username, p->pw_gecos, 0, ',');
-               uid = p->pw_uid;
-       } else {
-               uid = (-1);
+       {
+               struct passwd *p = (struct passwd *) getpwnam(username);
+
+               if (p != NULL) {
+                       extract_token(username, p->pw_gecos, 0, ',', sizeof username);
+                       uid = p->pw_uid;
+               } else {
+                       uid = (-1);
+               }
        }
 #else
        uid = (-1);
@@ -756,8 +805,8 @@ int create_user(char *newusername, int become_user)
        }
 
        /* Go ahead and initialize a new user record */
-       memset(&usbuf, 0, sizeof(struct user));
-       strcpy(usbuf.fullname, username);
+       memset(&usbuf, 0, sizeof(struct ctdluser));
+       safestrncpy(usbuf.fullname, username, sizeof usbuf.fullname);
        strcpy(usbuf.password, "");
        usbuf.uid = uid;
 
@@ -787,23 +836,26 @@ int create_user(char *newusername, int become_user)
         * Make the latter an invisible system room.
         */
        MailboxName(mailboxname, sizeof mailboxname, &usbuf, MAILROOM);
-       create_room(mailboxname, 5, "", 0, 1, 1);
+       create_room(mailboxname, 5, "", 0, 1, 1, VIEW_MAILBOX);
 
        MailboxName(mailboxname, sizeof mailboxname, &usbuf, USERCONFIGROOM);
-       create_room(mailboxname, 5, "", 0, 1, 1);
+       create_room(mailboxname, 5, "", 0, 1, 1, VIEW_BBS);
         if (lgetroom(&qrbuf, mailboxname) == 0) {
                 qrbuf.QRflags2 |= QR2_SYSTEM;
                 lputroom(&qrbuf);
         }
 
+       /* Perform any create functions registered by server extensions */
+       PerformUserHooks(&usbuf, EVT_NEWUSER);
+
        /* Everything below this line can be bypassed if administratively
-          creating a user, instead of doing self-service account creation
+        * creating a user, instead of doing self-service account creation
         */
 
        if (become_user) {
                /* Now become the user we just created */
-               memcpy(&CC->user, &usbuf, sizeof(struct user));
-               strcpy(CC->curr_user, username);
+               memcpy(&CC->user, &usbuf, sizeof(struct ctdluser));
+               safestrncpy(CC->curr_user, username, sizeof CC->curr_user);
                CC->logged_in = 1;
        
                /* Check to make sure we're still who we think we are */
@@ -812,7 +864,7 @@ int create_user(char *newusername, int become_user)
                }
        }
 
-       lprintf(3, "New user <%s> created\n", username);
+       lprintf(CTDL_NOTICE, "New user <%s> created\n", username);
        return (0);
 }
 
@@ -825,16 +877,16 @@ int create_user(char *newusername, int become_user)
 void cmd_newu(char *cmdbuf)
 {
        int a;
-       char username[SIZ];
+       char username[26];
 
        if (config.c_disable_newu) {
                cprintf("%d Self-service user account creation "
-                       "is disabled on this system.\n", ERROR);
+                       "is disabled on this system.\n", ERROR + NOT_HERE);
                return;
        }
 
        if (CC->logged_in) {
-               cprintf("%d Already logged in.\n", ERROR);
+               cprintf("%d Already logged in.\n", ERROR + ALREADY_LOGGED_IN);
                return;
        }
        if (CC->nologin) {
@@ -842,19 +894,19 @@ void cmd_newu(char *cmdbuf)
                        ERROR + MAX_SESSIONS_EXCEEDED,
                        config.c_nodename, config.c_maxsessions);
        }
-       extract(username, cmdbuf, 0);
+       extract_token(username, cmdbuf, 0, '|', sizeof username);
        username[25] = 0;
        strproc(username);
 
        if (strlen(username) == 0) {
-               cprintf("%d You must supply a user name.\n", ERROR);
+               cprintf("%d You must supply a user name.\n", ERROR + USERNAME_REQUIRED);
                return;
        }
 
        if ((!strcasecmp(username, "bbs")) ||
            (!strcasecmp(username, "new")) ||
            (!strcasecmp(username, "."))) {
-               cprintf("%d '%s' is an invalid login name.\n", ERROR, username);
+               cprintf("%d '%s' is an invalid login name.\n", ERROR + ILLEGAL_VALUE, username);
                return;
        }
 
@@ -872,7 +924,7 @@ void cmd_newu(char *cmdbuf)
                        ERROR + INTERNAL_ERROR);
                return;
        } else {
-               cprintf("%d unknown error\n", ERROR);
+               cprintf("%d unknown error\n", ERROR + INTERNAL_ERROR);
        }
 }
 
@@ -886,8 +938,8 @@ void cmd_setp(char *new_pw)
        if (CtdlAccessCheck(ac_logged_in)) {
                return;
        }
-       if ( (CC->user.uid != BBSUID) && (CC->user.uid != (-1)) ) {
-               cprintf("%d Not allowed.  Use the 'passwd' command.\n", ERROR);
+       if ( (CC->user.uid != CTDLUID) && (CC->user.uid != (-1)) ) {
+               cprintf("%d Not allowed.  Use the 'passwd' command.\n", ERROR + NOT_HERE);
                return;
        }
        strproc(new_pw);
@@ -896,46 +948,58 @@ void cmd_setp(char *new_pw)
                return;
        }
        lgetuser(&CC->user, CC->curr_user);
-       strcpy(CC->user.password, new_pw);
+       safestrncpy(CC->user.password, new_pw, sizeof(CC->user.password));
        lputuser(&CC->user);
        cprintf("%d Password changed.\n", CIT_OK);
-       lprintf(3, "Password changed for user <%s>\n", CC->curr_user);
+       lprintf(CTDL_INFO, "Password changed for user <%s>\n", CC->curr_user);
        PerformSessionHooks(EVT_SETPASS);
 }
 
 
 /*
- * cmd_creu()  -  administratively create a new user account (do not log in to it)
+ * cmd_creu() - administratively create a new user account (do not log in to it)
  */
 void cmd_creu(char *cmdbuf)
 {
        int a;
-       char username[SIZ];
+       char username[26];
+       char password[32];
+       struct ctdluser tmp;
 
        if (CtdlAccessCheck(ac_aide)) {
                return;
        }
 
-       extract(username, cmdbuf, 0);
+       extract_token(username, cmdbuf, 0, '|', sizeof username);
+       extract_token(password, cmdbuf, 1, '|', sizeof password);
        username[25] = 0;
+       password[31] = 0;
        strproc(username);
+       strproc(password);
 
        if (strlen(username) == 0) {
-               cprintf("%d You must supply a user name.\n", ERROR);
+               cprintf("%d You must supply a user name.\n", ERROR + USERNAME_REQUIRED);
                return;
        }
 
        a = create_user(username, 0);
 
        if (a == 0) {
-               cprintf("%d ok\n", CIT_OK);
+               if (strlen(password) > 0) {
+                       lgetuser(&tmp, username);
+                       safestrncpy(tmp.password, password, sizeof(tmp.password));
+                       lputuser(&tmp);
+               }
+               cprintf("%d User '%s' created %s.\n", CIT_OK, username,
+                               (strlen(password) > 0) ? "and password set" :
+                               "with no password");
                return;
        } else if (a == ERROR + ALREADY_EXISTS) {
                cprintf("%d '%s' already exists.\n",
                        ERROR + ALREADY_EXISTS, username);
                return;
        } else {
-               cprintf("%d An error occured creating the user account.\n", ERROR);
+               cprintf("%d An error occured creating the user account.\n", ERROR + INTERNAL_ERROR);
        }
 }
 
@@ -968,7 +1032,7 @@ void cmd_setu(char *new_parms)
                return;
 
        if (num_parms(new_parms) < 3) {
-               cprintf("%d Usage error.\n", ERROR);
+               cprintf("%d Usage error.\n", ERROR + ILLEGAL_VALUE);
                return;
        }
        lgetuser(&CC->user, CC->curr_user);
@@ -989,6 +1053,7 @@ void cmd_slrp(char *new_ptr)
 {
        long newlr;
        struct visit vbuf;
+       struct visit original_vbuf;
 
        if (CtdlAccessCheck(ac_logged_in)) {
                return;
@@ -1003,9 +1068,15 @@ void cmd_slrp(char *new_ptr)
        lgetuser(&CC->user, CC->curr_user);
 
        CtdlGetRelationship(&vbuf, &CC->user, &CC->room);
+       memcpy(&original_vbuf, &vbuf, sizeof(struct visit));
        vbuf.v_lastseen = newlr;
        snprintf(vbuf.v_seen, sizeof vbuf.v_seen, "*:%ld", newlr);
-       CtdlSetRelationship(&vbuf, &CC->user, &CC->room);
+
+       /* Only rewrite the record if it changed */
+       if ( (vbuf.v_lastseen != original_vbuf.v_lastseen)
+          || (strcmp(vbuf.v_seen, original_vbuf.v_seen)) ) {
+               CtdlSetRelationship(&vbuf, &CC->user, &CC->room);
+       }
 
        lputuser(&CC->user);
        cprintf("%d %ld\n", CIT_OK, newlr);
@@ -1021,14 +1092,14 @@ void cmd_seen(char *argbuf) {
        }
 
        if (num_parms(argbuf) != 2) {
-               cprintf("%d Invalid parameters\n", ERROR);
+               cprintf("%d Invalid parameters\n", ERROR + ILLEGAL_VALUE);
                return;
        }
 
        target_msgnum = extract_long(argbuf, 0);
        target_setting = extract_int(argbuf, 1);
 
-       CtdlSetSeen(target_msgnum, target_setting);
+       CtdlSetSeen(target_msgnum, target_setting, ctdlsetseen_seen);
        cprintf("%d OK\n", CIT_OK);
 }
 
@@ -1040,21 +1111,53 @@ void cmd_gtsn(char *argbuf) {
                return;
        }
 
-       CtdlGetSeen(buf);
+       CtdlGetSeen(buf, ctdlsetseen_seen);
        cprintf("%d %s\n", CIT_OK, buf);
 }
 
 
+/*
+ * API function for cmd_invt_kick() and anything else that needs to
+ * invite or kick out a user to/from a room.
+ * 
+ * Set iuser to the name of the user, and op to 1=invite or 0=kick
+ */
+int CtdlInvtKick(char *iuser, int op) {
+       struct ctdluser USscratch;
+       struct visit vbuf;
+       char bbb[SIZ];
+
+       if (getuser(&USscratch, iuser) != 0) {
+               return(1);
+       }
+
+       CtdlGetRelationship(&vbuf, &USscratch, &CC->room);
+       if (op == 1) {
+               vbuf.v_flags = vbuf.v_flags & ~V_FORGET & ~V_LOCKOUT;
+               vbuf.v_flags = vbuf.v_flags | V_ACCESS;
+       }
+       if (op == 0) {
+               vbuf.v_flags = vbuf.v_flags & ~V_ACCESS;
+               vbuf.v_flags = vbuf.v_flags | V_FORGET | V_LOCKOUT;
+       }
+       CtdlSetRelationship(&vbuf, &USscratch, &CC->room);
+
+       /* post a message in Aide> saying what we just did */
+       snprintf(bbb, sizeof bbb, "%s %s %s> by %s\n",
+               iuser,
+               ((op == 1) ? "invited to" : "kicked out of"),
+               CC->room.QRname,
+               CC->user.fullname);
+       aide_message(bbb);
+
+       return(0);
+}
+
 
 /*
  * INVT and KICK commands
  */
-void cmd_invt_kick(char *iuser, int op)
-                       /* user name */
-{                              /* 1 = invite, 0 = kick out */
-       struct user USscratch;
-       char bbb[SIZ];
-       struct visit vbuf;
+void cmd_invt_kick(char *iuser, int op) {
 
        /*
         * These commands are only allowed by aides, room aides,
@@ -1077,31 +1180,10 @@ void cmd_invt_kick(char *iuser, int op)
                return;
        }
 
-       if (lgetuser(&USscratch, iuser) != 0) {
-               cprintf("%d No such user.\n", ERROR);
+       if (CtdlInvtKick(iuser, op) != 0) {
+               cprintf("%d No such user.\n", ERROR + NO_SUCH_USER);
                return;
        }
-       CtdlGetRelationship(&vbuf, &USscratch, &CC->room);
-
-       if (op == 1) {
-               vbuf.v_flags = vbuf.v_flags & ~V_FORGET & ~V_LOCKOUT;
-               vbuf.v_flags = vbuf.v_flags | V_ACCESS;
-       }
-       if (op == 0) {
-               vbuf.v_flags = vbuf.v_flags & ~V_ACCESS;
-               vbuf.v_flags = vbuf.v_flags | V_FORGET | V_LOCKOUT;
-       }
-       CtdlSetRelationship(&vbuf, &USscratch, &CC->room);
-
-       lputuser(&USscratch);
-
-       /* post a message in Aide> saying what we just did */
-       snprintf(bbb, sizeof bbb, "%s %s %s> by %s\n",
-               iuser,
-               ((op == 1) ? "invited to" : "kicked out of"),
-               CC->room.QRname,
-               CC->user.fullname);
-       aide_message(bbb);
 
        cprintf("%d %s %s %s.\n",
                CIT_OK, iuser,
@@ -1154,7 +1236,7 @@ void cmd_forg(void)
                cprintf("%d Ok\n", CIT_OK);
        }
        else {
-               cprintf("%d You may not forget this room.\n", ERROR);
+               cprintf("%d You may not forget this room.\n", ERROR + NOT_HERE);
        }
 }
 
@@ -1164,7 +1246,7 @@ void cmd_forg(void)
 void cmd_gnur(void)
 {
        struct cdbdata *cdbus;
-       struct user usbuf;
+       struct ctdluser usbuf;
 
        if (CtdlAccessCheck(ac_aide)) {
                return;
@@ -1178,17 +1260,17 @@ void cmd_gnur(void)
        /* There are unvalidated users.  Traverse the user database,
         * and return the first user we find that needs validation.
         */
-       cdb_rewind(CDB_USERSUPP);
-       while (cdbus = cdb_next_item(CDB_USERSUPP), cdbus != NULL) {
-               memset(&usbuf, 0, sizeof(struct user));
+       cdb_rewind(CDB_USERS);
+       while (cdbus = cdb_next_item(CDB_USERS), cdbus != NULL) {
+               memset(&usbuf, 0, sizeof(struct ctdluser));
                memcpy(&usbuf, cdbus->ptr,
-                      ((cdbus->len > sizeof(struct user)) ?
-                       sizeof(struct user) : cdbus->len));
+                      ((cdbus->len > sizeof(struct ctdluser)) ?
+                       sizeof(struct ctdluser) : cdbus->len));
                cdb_free(cdbus);
                if ((usbuf.flags & US_NEEDVALID)
                    && (usbuf.axlevel > 0)) {
                        cprintf("%d %s\n", MORE_DATA, usbuf.fullname);
-                       cdb_close_cursor(CDB_USERSUPP);
+                       cdb_close_cursor(CDB_USERS);
                        return;
                }
        }
@@ -1213,11 +1295,11 @@ void cmd_gnur(void)
  */
 void cmd_vali(char *v_args)
 {
-       char user[SIZ];
+       char user[128];
        int newax;
-       struct user userbuf;
+       struct ctdluser userbuf;
 
-       extract(user, v_args, 0);
+       extract_token(user, v_args, 0, '|', sizeof user);
        newax = extract_int(v_args, 1);
 
        if (CtdlAccessCheck(ac_aide)) {
@@ -1249,19 +1331,19 @@ void cmd_vali(char *v_args)
 /* 
  *  Traverse the user file...
  */
-void ForEachUser(void (*CallBack) (struct user * EachUser, void *out_data),
+void ForEachUser(void (*CallBack) (struct ctdluser * EachUser, void *out_data),
                 void *in_data)
 {
-       struct user usbuf;
+       struct ctdluser usbuf;
        struct cdbdata *cdbus;
 
-       cdb_rewind(CDB_USERSUPP);
+       cdb_rewind(CDB_USERS);
 
-       while (cdbus = cdb_next_item(CDB_USERSUPP), cdbus != NULL) {
-               memset(&usbuf, 0, sizeof(struct user));
+       while (cdbus = cdb_next_item(CDB_USERS), cdbus != NULL) {
+               memset(&usbuf, 0, sizeof(struct ctdluser));
                memcpy(&usbuf, cdbus->ptr,
-                      ((cdbus->len > sizeof(struct user)) ?
-                       sizeof(struct user) : cdbus->len));
+                      ((cdbus->len > sizeof(struct ctdluser)) ?
+                       sizeof(struct ctdluser) : cdbus->len));
                cdb_free(cdbus);
                (*CallBack) (&usbuf, in_data);
        }
@@ -1271,7 +1353,7 @@ void ForEachUser(void (*CallBack) (struct user * EachUser, void *out_data),
 /*
  * List one user (this works with cmd_list)
  */
-void ListThisUser(struct user *usbuf, void *data)
+void ListThisUser(struct ctdluser *usbuf, void *data)
 {
        if (usbuf->axlevel > 0) {
                if ((CC->user.axlevel >= 6)
@@ -1339,7 +1421,7 @@ void cmd_chek(void)
  */
 void cmd_qusr(char *who)
 {
-       struct user usbuf;
+       struct ctdluser usbuf;
 
        if (getuser(&usbuf, who) == 0) {
                cprintf("%d %s\n", CIT_OK, usbuf.fullname);
@@ -1354,14 +1436,14 @@ void cmd_qusr(char *who)
  */
 void cmd_agup(char *cmdbuf)
 {
-       struct user usbuf;
-       char requested_user[SIZ];
+       struct ctdluser usbuf;
+       char requested_user[128];
 
        if (CtdlAccessCheck(ac_aide)) {
                return;
        }
 
-       extract(requested_user, cmdbuf, 0);
+       extract_token(requested_user, cmdbuf, 0, '|', sizeof requested_user);
        if (getuser(&usbuf, requested_user) != 0) {
                cprintf("%d No such user.\n", ERROR + NO_SUCH_USER);
                return;
@@ -1386,8 +1468,8 @@ void cmd_agup(char *cmdbuf)
  */
 void cmd_asup(char *cmdbuf)
 {
-       struct user usbuf;
-       char requested_user[SIZ];
+       struct ctdluser usbuf;
+       char requested_user[128];
        char notify[SIZ];
        int np;
        int newax;
@@ -1396,14 +1478,14 @@ void cmd_asup(char *cmdbuf)
        if (CtdlAccessCheck(ac_aide))
                return;
 
-       extract(requested_user, cmdbuf, 0);
+       extract_token(requested_user, cmdbuf, 0, '|', sizeof requested_user);
        if (lgetuser(&usbuf, requested_user) != 0) {
                cprintf("%d No such user.\n", ERROR + NO_SUCH_USER);
                return;
        }
        np = num_parms(cmdbuf);
        if (np > 1)
-               extract(usbuf.password, cmdbuf, 1);
+               extract_token(usbuf.password, cmdbuf, 1, '|', sizeof usbuf.password);
        if (np > 2)
                usbuf.flags = extract_int(cmdbuf, 2);
        if (np > 3)
@@ -1485,7 +1567,7 @@ int InitialMailCheck()
         int num_newmsgs = 0;
         int a;
         char mailboxname[ROOMNAMELEN];
-        struct room mailbox;
+        struct ctdlroom mailbox;
         struct visit vbuf;
         struct cdbdata *cdbfr;
         long *msglist = NULL;
@@ -1499,7 +1581,7 @@ int InitialMailCheck()
         cdbfr = cdb_fetch(CDB_MSGLISTS, &mailbox.QRnumber, sizeof(long));
 
         if (cdbfr != NULL) {
-                msglist = mallok(cdbfr->len);
+                msglist = malloc(cdbfr->len);
                 memcpy(msglist, cdbfr->ptr, cdbfr->len);
                 num_msgs = cdbfr->len / sizeof(long);
                 cdb_free(cdbfr);
@@ -1513,7 +1595,7 @@ int InitialMailCheck()
                         }
                 }
         if (msglist != NULL)
-                phree(msglist);
+                free(msglist);
 
         return (num_newmsgs);
 }