X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=citadel%2Fuser_ops.c;h=91e3d077a56934c3c72b4f0ef6d4feb1071d61d7;hb=fc86ffaec2bf78ee472ed8fffc4f0a0fa9599105;hp=26bcb0c7205fe596ab5302616f6291bdb0fd94b7;hpb=73a4e2f4e3f8ae5d9f3216eeee58d8c69eac04b5;p=citadel.git

diff --git a/citadel/user_ops.c b/citadel/user_ops.c
index 26bcb0c72..91e3d077a 100644
--- a/citadel/user_ops.c
+++ b/citadel/user_ops.c
@@ -84,6 +84,16 @@ INLINE void makeuserkey(char *key, char *username) {
  *	       returns 0 on success
  */
 int getuser(struct ctdluser *usbuf, char name[])
+{
+	return CtdlGetUser(usbuf, name);
+}
+
+
+/*
+ * CtdlGetUser()  -  retrieve named user into supplied buffer.
+ *	       returns 0 on success
+ */
+int CtdlGetUser(struct ctdluser *usbuf, char name[])
 {
 
 	char usernamekey[USERNAME_SIZE];
@@ -111,13 +121,13 @@ int getuser(struct ctdluser *usbuf, char name[])
 
 
 /*
- * lgetuser()  -  same as getuser() but locks the record
+ * CtdlGetUserLock()  -  same as getuser() but locks the record
  */
-int lgetuser(struct ctdluser *usbuf, char *name)
+int CtdlGetUserLock(struct ctdluser *usbuf, char *name)
 {
 	int retcode;
 
-	retcode = getuser(usbuf, name);
+	retcode = CtdlGetUser(usbuf, name);
 	if (retcode == 0) {
 		begin_critical_section(S_USERS);
 	}
@@ -126,9 +136,18 @@ int lgetuser(struct ctdluser *usbuf, char *name)
 
 
 /*
- * putuser()  -  write user buffer into the correct place on disk
+ * lgetuser()  -  same as getuser() but locks the record
  */
-void putuser(struct ctdluser *usbuf)
+int lgetuser(struct ctdluser *usbuf, char *name)
+{
+	return CtdlGetUserLock(usbuf, name);
+}
+
+
+/*
+ * CtdlPutUser()  -  write user buffer into the correct place on disk
+ */
+void CtdlPutUser(struct ctdluser *usbuf)
 {
 	char usernamekey[USERNAME_SIZE];
 
@@ -142,13 +161,31 @@ void putuser(struct ctdluser *usbuf)
 }
 
 
+/*
+ * putuser()  -  write user buffer into the correct place on disk
+ */
+void putuser(struct ctdluser *usbuf)
+{
+	CtdlPutUser(usbuf);
+}
+
+
+/*
+ * CtdlPutUserLock()  -  same as putuser() but locks the record
+ */
+void CtdlPutUserLock(struct ctdluser *usbuf)
+{
+	CtdlPutUser(usbuf);
+	end_critical_section(S_USERS);
+}
+
+
 /*
  * lputuser()  -  same as putuser() but locks the record
  */
 void lputuser(struct ctdluser *usbuf)
 {
-	putuser(usbuf);
-	end_critical_section(S_USERS);
+	CtdlPutUserLock(usbuf);
 }
 
 
@@ -180,12 +217,12 @@ int rename_user(char *oldname, char *newname) {
 	/* Lock up and get going */
 	begin_critical_section(S_USERS);
 
-	if (getuser(&usbuf, newname) == 0) {
+	if (CtdlGetUser(&usbuf, newname) == 0) {
 		retcode = RENAMEUSER_ALREADY_EXISTS;
 	}
 	else {
 
-		if (getuser(&usbuf, oldname) != 0) {
+		if (CtdlGetUser(&usbuf, oldname) != 0) {
 			retcode = RENAMEUSER_NOT_FOUND;
 		}
 
@@ -198,7 +235,7 @@ int rename_user(char *oldname, char *newname) {
 				CtdlLogPrintf(CTDL_DEBUG, "Renaming <%s> to <%s>\n", oldname, newname);
 				cdb_delete(CDB_USERS, oldnamekey, strlen(oldnamekey));
 				safestrncpy(usbuf.fullname, newname, sizeof usbuf.fullname);
-				putuser(&usbuf);
+				CtdlPutUser(&usbuf);
 				cdb_store(CDB_USERSBYNUMBER, &usbuf.usernum, sizeof(long),
 					usbuf.fullname, strlen(usbuf.fullname)+1 );
 
@@ -324,6 +361,12 @@ void CtdlGetRelationship(struct visit *vbuf,
 }
 
 
+void CtdlMailboxName(char *buf, size_t n, const struct ctdluser *who, const char *prefix)
+{
+	snprintf(buf, n, "%010ld.%s", who->usernum, prefix);
+}
+
+
 void MailboxName(char *buf, size_t n, const struct ctdluser *who, const char *prefix)
 {
 	snprintf(buf, n, "%010ld.%s", who->usernum, prefix);
@@ -361,12 +404,12 @@ int is_room_aide(void)
 }
 
 /*
- * getuserbynumber() -	get user by number
+ * CtdlGetUserByNumber() -	get user by number
  * 			returns 0 if user was found
  *
  * Note: fetching a user this way requires one additional database operation.
  */
-int getuserbynumber(struct ctdluser *usbuf, long number)
+int CtdlGetUserByNumber(struct ctdluser *usbuf, long number)
 {
 	struct cdbdata *cdbun;
 	int r;
@@ -378,11 +421,22 @@ int getuserbynumber(struct ctdluser *usbuf, long number)
 	}
 
 	CtdlLogPrintf(CTDL_INFO, "User %ld maps to %s\n", number, cdbun->ptr);
-	r = getuser(usbuf, cdbun->ptr);
+	r = CtdlGetUser(usbuf, cdbun->ptr);
 	cdb_free(cdbun);
 	return(r);
 }
 
+/*
+ * getuserbynumber() -	get user by number
+ * 			returns 0 if user was found
+ *
+ * Note: fetching a user this way requires one additional database operation.
+ */
+int getuserbynumber(struct ctdluser *usbuf, long number)
+{
+	return CtdlGetUserByNumber(usbuf, number);
+}
+
 
 
 /*
@@ -578,7 +632,7 @@ int CtdlLoginExistingUser(char *authname, char *trythisname)
 		struct recptypes *valid = NULL;
 	
 		/* First, try to log in as if the supplied name is a display name */
-		found_user = getuser(&CC->user, username);
+		found_user = CtdlGetUser(&CC->user, username);
 	
 		/* If that didn't work, try to log in as if the supplied name
 	 	* is an e-mail address
@@ -587,7 +641,7 @@ int CtdlLoginExistingUser(char *authname, char *trythisname)
 			valid = validate_recipients(username, NULL, 0);
 			if (valid != NULL) {
 				if (valid->num_local == 1) {
-					found_user = getuser(&CC->user, valid->recp_local);
+					found_user = CtdlGetUser(&CC->user, valid->recp_local);
 				}
 				free_recipients(valid);
 			}
@@ -657,7 +711,7 @@ void do_login(void)
 	CC->logged_in = 1;
 	CtdlLogPrintf(CTDL_NOTICE, "<%s> logged in\n", CC->curr_user);
 
-	lgetuser(&CC->user, CC->curr_user);
+	CtdlGetUserLock(&CC->user, CC->curr_user);
 	++(CC->user.timescalled);
 	CC->previous_login = CC->user.lastcall;
 	time(&CC->user.lastcall);
@@ -678,7 +732,7 @@ void do_login(void)
 		}
 	}
 
-	lputuser(&CC->user);
+	CtdlPutUserLock(&CC->user);
 
 	/*
 	 * Populate CC->cs_inet_email with a default address.  This will be
@@ -692,16 +746,16 @@ void do_login(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, VIEW_MAILBOX);
-	create_room(SENTITEMS, 4, "", 0, 1, 0, VIEW_MAILBOX);
-	create_room(USERTRASHROOM, 4, "", 0, 1, 0, VIEW_MAILBOX);
-	/* create_room(USERDRAFTROOM, 4, "", 0, 1, 0, VIEW_MAILBOX); temporarily disabled for 7.60 */
+	CtdlCreateRoom(MAILROOM, 4, "", 0, 1, 0, VIEW_MAILBOX);
+	CtdlCreateRoom(SENTITEMS, 4, "", 0, 1, 0, VIEW_MAILBOX);
+	CtdlCreateRoom(USERTRASHROOM, 4, "", 0, 1, 0, VIEW_MAILBOX);
+	/* CtdlCreateRoom(USERDRAFTROOM, 4, "", 0, 1, 0, VIEW_MAILBOX); temporarily disabled for 7.60 */
 
 	/* Run any startup routines registered by loadable modules */
 	PerformSessionHooks(EVT_LOGIN);
 
 	/* Enter the lobby */
-	usergoto(config.c_baseroom, 0, 0, NULL, NULL);
+	CtdlUserGoto(config.c_baseroom, 0, 0, NULL, NULL);
 }
 
 
@@ -772,6 +826,7 @@ void logout(void)
 static int validpw(uid_t uid, const char *pass)
 {
 	char buf[256];
+	int rv = 0;
 
 	if (IsEmptyStr(pass)) {
 		CtdlLogPrintf(CTDL_DEBUG, "refusing to check empty password for uid=%d using chkpwd...\n", uid);
@@ -781,9 +836,9 @@ static int validpw(uid_t uid, const char *pass)
 	CtdlLogPrintf(CTDL_DEBUG, "Validating password for uid=%d using chkpwd...\n", uid);
 
 	begin_critical_section(S_CHKPWD);
-	write(chkpwd_write_pipe[1], &uid, sizeof(uid_t));
-	write(chkpwd_write_pipe[1], pass, 256);
-	read(chkpwd_read_pipe[0], buf, 4);
+	rv = write(chkpwd_write_pipe[1], &uid, sizeof(uid_t));
+	rv = write(chkpwd_write_pipe[1], pass, 256);
+	rv = read(chkpwd_read_pipe[0], buf, 4);
 	end_critical_section(S_CHKPWD);
 
 	if (!strncmp(buf, "PASS", 4)) {
@@ -852,7 +907,7 @@ int CtdlTryPassword(char *password)
 		CtdlLogPrintf(CTDL_WARNING, "CtdlTryPassword: no user selected\n");
 		return pass_no_user;
 	}
-	if (getuser(&CC->user, CC->curr_user)) {
+	if (CtdlGetUser(&CC->user, CC->curr_user)) {
 		CtdlLogPrintf(CTDL_ERR, "CtdlTryPassword: internal error\n");
 		return pass_internal_error;
 	}
@@ -881,9 +936,9 @@ int CtdlTryPassword(char *password)
 			 * this is a security hazard, comment it out.
 			 */
 
-			lgetuser(&CC->user, CC->curr_user);
+			CtdlGetUserLock(&CC->user, CC->curr_user);
 			safestrncpy(CC->user.password, password, sizeof CC->user.password);
-			lputuser(&CC->user);
+			CtdlPutUserLock(&CC->user);
 
 			/*
 			 * (sooper-seekrit hack ends here)
@@ -976,7 +1031,7 @@ int purge_user(char pname[])
 	if (IsEmptyStr(pname))
 		return (ERROR + NO_SUCH_USER);
 
-	if (getuser(&usbuf, pname) != 0) {
+	if (CtdlGetUser(&usbuf, pname) != 0) {
 		CtdlLogPrintf(CTDL_ERR, "Cannot purge user <%s> - not found\n", pname);
 		return (ERROR + NO_SUCH_USER);
 	}
@@ -995,7 +1050,7 @@ int purge_user(char pname[])
 	if (user_is_logged_in == 1) {
 		CtdlLogPrintf(CTDL_WARNING, "User <%s> is logged in; not deleting.\n", pname);
 		usbuf.axlevel = 0;
-		putuser(&usbuf);
+		CtdlPutUser(&usbuf);
 		return (1);
 	}
 	CtdlLogPrintf(CTDL_NOTICE, "Deleting user <%s>\n", pname);
@@ -1034,7 +1089,7 @@ int purge_user(char pname[])
 
 int internal_create_user (char *username, struct ctdluser *usbuf, uid_t uid)
 {
-	if (!getuser(usbuf, username)) {
+	if (!CtdlGetUser(usbuf, username)) {
 		return (ERROR + ALREADY_EXISTS);
 	}
 
@@ -1058,7 +1113,7 @@ int internal_create_user (char *username, struct ctdluser *usbuf, uid_t uid)
 	usbuf->usernum = get_new_user_number();
 
 	/* add user to the database */
-	putuser(usbuf);
+	CtdlPutUser(usbuf);
 	cdb_store(CDB_USERSBYNUMBER, &usbuf->usernum, sizeof(long), usbuf->fullname, strlen(usbuf->fullname)+1);
 
 	return 0;
@@ -1133,14 +1188,14 @@ int create_user(char *newusername, int become_user)
 	 * Give the user a private mailbox and a configuration room.
 	 * Make the latter an invisible system room.
 	 */
-	MailboxName(mailboxname, sizeof mailboxname, &usbuf, MAILROOM);
-	create_room(mailboxname, 5, "", 0, 1, 1, VIEW_MAILBOX);
+	CtdlMailboxName(mailboxname, sizeof mailboxname, &usbuf, MAILROOM);
+	CtdlCreateRoom(mailboxname, 5, "", 0, 1, 1, VIEW_MAILBOX);
 
-	MailboxName(mailboxname, sizeof mailboxname, &usbuf, USERCONFIGROOM);
-	create_room(mailboxname, 5, "", 0, 1, 1, VIEW_BBS);
-	if (lgetroom(&qrbuf, mailboxname) == 0) {
+	CtdlMailboxName(mailboxname, sizeof mailboxname, &usbuf, USERCONFIGROOM);
+	CtdlCreateRoom(mailboxname, 5, "", 0, 1, 1, VIEW_BBS);
+	if (CtdlGetRoomLock(&qrbuf, mailboxname) == 0) {
 		qrbuf.QRflags2 |= QR2_SYSTEM;
-		lputroom(&qrbuf);
+		CtdlPutRoomLock(&qrbuf);
 	}
 
 	/* Perform any create functions registered by server extensions */
@@ -1157,7 +1212,7 @@ int create_user(char *newusername, int become_user)
 		do_login();
 	
 		/* Check to make sure we're still who we think we are */
-		if (getuser(&CC->user, CC->curr_user)) {
+		if (CtdlGetUser(&CC->user, CC->curr_user)) {
 			return (ERROR + INTERNAL_ERROR);
 		}
 	}
@@ -1168,7 +1223,7 @@ int create_user(char *newusername, int become_user)
 		CC->cs_host,
 		CC->cs_addr
 	);
-	aide_message(buf, "User Creation Notice");
+	CtdlAideMessage(buf, "User Creation Notice");
 	CtdlLogPrintf(CTDL_NOTICE, "New user <%s> created\n", username);
 	return (0);
 }
@@ -1243,9 +1298,9 @@ void cmd_newu(char *cmdbuf)
  */
 void CtdlSetPassword(char *new_pw)
 {
-	lgetuser(&CC->user, CC->curr_user);
+	CtdlGetUserLock(&CC->user, CC->curr_user);
 	safestrncpy(CC->user.password, new_pw, sizeof(CC->user.password));
-	lputuser(&CC->user);
+	CtdlPutUserLock(&CC->user);
 	CtdlLogPrintf(CTDL_INFO, "Password changed for user <%s>\n", CC->curr_user);
 	PerformSessionHooks(EVT_SETPASS);
 }
@@ -1320,9 +1375,9 @@ void cmd_creu(char *cmdbuf)
 
 	if (a == 0) {
 		if (!IsEmptyStr(password)) {
-			lgetuser(&tmp, username);
+			CtdlGetUserLock(&tmp, username);
 			safestrncpy(tmp.password, password, sizeof(tmp.password));
-			lputuser(&tmp);
+			CtdlPutUserLock(&tmp);
 		}
 		cprintf("%d User '%s' created %s.\n", CIT_OK, username,
 				(!IsEmptyStr(password)) ? "and password set" :
@@ -1351,7 +1406,7 @@ void cmd_getu(char *cmdbuf)
 	if (CtdlAccessCheck(ac_logged_in))
 		return;
 
-	getuser(&CC->user, CC->curr_user);
+	CtdlGetUser(&CC->user, CC->curr_user);
 	cprintf("%d %d|%d|%d|\n",
 		CIT_OK,
 		CC->user.USscreenwidth,
@@ -1372,14 +1427,14 @@ void cmd_setu(char *new_parms)
 		cprintf("%d Usage error.\n", ERROR + ILLEGAL_VALUE);
 		return;
 	}
-	lgetuser(&CC->user, CC->curr_user);
+	CtdlGetUserLock(&CC->user, CC->curr_user);
 	CC->user.USscreenwidth = extract_int(new_parms, 0);
 	CC->user.USscreenheight = extract_int(new_parms, 1);
 	CC->user.flags = CC->user.flags & (~US_USER_SET);
 	CC->user.flags = CC->user.flags |
 	    (extract_int(new_parms, 2) & US_USER_SET);
 
-	lputuser(&CC->user);
+	CtdlPutUserLock(&CC->user);
 	cprintf("%d Ok\n", CIT_OK);
 }
 
@@ -1402,7 +1457,7 @@ void cmd_slrp(char *new_ptr)
 		newlr = atol(new_ptr);
 	}
 
-	lgetuser(&CC->user, CC->curr_user);
+	CtdlGetUserLock(&CC->user, CC->curr_user);
 
 	CtdlGetRelationship(&vbuf, &CC->user, &CC->room);
 	memcpy(&original_vbuf, &vbuf, sizeof(struct visit));
@@ -1415,7 +1470,7 @@ void cmd_slrp(char *new_ptr)
 		CtdlSetRelationship(&vbuf, &CC->user, &CC->room);
 	}
 
-	lputuser(&CC->user);
+	CtdlPutUserLock(&CC->user);
 	cprintf("%d %ld\n", CIT_OK, newlr);
 }
 
@@ -1465,7 +1520,7 @@ int CtdlInvtKick(char *iuser, int op) {
 	struct visit vbuf;
 	char bbb[SIZ];
 
-	if (getuser(&USscratch, iuser) != 0) {
+	if (CtdlGetUser(&USscratch, iuser) != 0) {
 		return(1);
 	}
 
@@ -1486,7 +1541,7 @@ int CtdlInvtKick(char *iuser, int op) {
 		((op == 1) ? "invited to" : "kicked out of"),
 		CC->room.QRname,
 		CC->user.fullname);
-	aide_message(bbb,"User Admin Message");
+	CtdlAideMessage(bbb,"User Admin Message");
 
 	return(0);
 }
@@ -1546,17 +1601,17 @@ int CtdlForgetThisRoom(void) {
 		return(1);
 	}
 
-	lgetuser(&CC->user, CC->curr_user);
+	CtdlGetUserLock(&CC->user, CC->curr_user);
 	CtdlGetRelationship(&vbuf, &CC->user, &CC->room);
 
 	vbuf.v_flags = vbuf.v_flags | V_FORGET;
 	vbuf.v_flags = vbuf.v_flags & ~V_ACCESS;
 
 	CtdlSetRelationship(&vbuf, &CC->user, &CC->room);
-	lputuser(&CC->user);
+	CtdlPutUserLock(&CC->user);
 
 	/* Return to the Lobby, so we don't end up in an undefined room */
-	usergoto(config.c_baseroom, 0, 0, NULL, NULL);
+	CtdlUserGoto(config.c_baseroom, 0, 0, NULL, NULL);
 	return(0);
 
 }
@@ -1646,7 +1701,7 @@ void cmd_vali(char *v_args)
 		return;
 	}
 
-	if (lgetuser(&userbuf, user) != 0) {
+	if (CtdlGetUserLock(&userbuf, user) != 0) {
 		cprintf("%d '%s' not found.\n", ERROR + NO_SUCH_USER, user);
 		return;
 	}
@@ -1654,7 +1709,7 @@ void cmd_vali(char *v_args)
 	userbuf.axlevel = newax;
 	userbuf.flags = (userbuf.flags & ~US_NEEDVALID);
 
-	lputuser(&userbuf);
+	CtdlPutUserLock(&userbuf);
 
 	/* If the access level was set to zero, delete the user */
 	if (newax == 0) {
@@ -1746,7 +1801,7 @@ void cmd_chek(char *argbuf)
 		return;
 	}
 
-	getuser(&CC->user, CC->curr_user);	/* no lock is needed here */
+	CtdlGetUser(&CC->user, CC->curr_user);	/* no lock is needed here */
 	if ((REGISCALL != 0) && ((CC->user.flags & US_REGIS) == 0))
 		regis = 1;
 
@@ -1770,7 +1825,7 @@ void cmd_qusr(char *who)
 {
 	struct ctdluser usbuf;
 
-	if (getuser(&usbuf, who) == 0) {
+	if (CtdlGetUser(&usbuf, who) == 0) {
 		cprintf("%d %s\n", CIT_OK, usbuf.fullname);
 	} else {
 		cprintf("%d No such user.\n", ERROR + NO_SUCH_USER);
@@ -1791,7 +1846,7 @@ void cmd_agup(char *cmdbuf)
 	}
 
 	extract_token(requested_user, cmdbuf, 0, '|', sizeof requested_user);
-	if (getuser(&usbuf, requested_user) != 0) {
+	if (CtdlGetUser(&usbuf, requested_user) != 0) {
 		cprintf("%d No such user.\n", ERROR + NO_SUCH_USER);
 		return;
 	}
@@ -1826,7 +1881,7 @@ void cmd_asup(char *cmdbuf)
 		return;
 
 	extract_token(requested_user, cmdbuf, 0, '|', sizeof requested_user);
-	if (lgetuser(&usbuf, requested_user) != 0) {
+	if (CtdlGetUserLock(&usbuf, requested_user) != 0) {
 		cprintf("%d No such user.\n", ERROR + NO_SUCH_USER);
 		return;
 	}
@@ -1851,7 +1906,7 @@ void cmd_asup(char *cmdbuf)
 	if (np > 8) {
 		usbuf.USuserpurge = extract_int(cmdbuf, 8);
 	}
-	lputuser(&usbuf);
+	CtdlPutUserLock(&usbuf);
 	if (usbuf.axlevel == 0) {
 		if (purge_user(requested_user) == 0) {
 			deleted = 1;
@@ -1862,7 +1917,7 @@ void cmd_asup(char *cmdbuf)
 		snprintf(notify, SIZ, 
 			 "User \"%s\" has been deleted by %s.\n",
 			 usbuf.fullname, CC->user.fullname);
-		aide_message(notify, "User Deletion Message");
+		CtdlAideMessage(notify, "User Deletion Message");
 	}
 
 	cprintf("%d Ok", CIT_OK);
@@ -1921,8 +1976,8 @@ int InitialMailCheck()
 	long *msglist = NULL;
 	int num_msgs = 0;
 
-	MailboxName(mailboxname, sizeof mailboxname, &CC->user, MAILROOM);
-	if (getroom(&mailbox, mailboxname) != 0)
+	CtdlMailboxName(mailboxname, sizeof mailboxname, &CC->user, MAILROOM);
+	if (CtdlGetRoom(&mailbox, mailboxname) != 0)
 		return (0);
 	CtdlGetRelationship(&vbuf, &CC->user, &mailbox);