Remove CTDLUID definition, point to ctdluid symbol instead.
authorArt Cancro <ajc@citadel.org>
Wed, 23 Aug 2023 18:37:10 +0000 (09:37 -0900)
committerArt Cancro <ajc@citadel.org>
Wed, 23 Aug 2023 18:37:10 +0000 (09:37 -0900)
CTDLUID was probably a hard-coded preprocessor definition back in the
ancient times.  Now it is not.

citadel/server/citserver.c
citadel/server/ctdl_module.h
citadel/server/database.c
citadel/server/modules/ctdlproto/serv_user.c
citadel/server/server_main.c

index b16235cb3b9ae0de7c933973e73e79c1eaa9ee08..73ca220b5045eb1b0df5519f318da77429ed1a58 100644 (file)
@@ -67,7 +67,7 @@ void master_startup(void) {
                gid = pw->pw_gid;
        }
 
-       if (create_run_directories(CTDLUID, gid) != 0) {
+       if (create_run_directories(ctdluid, gid) != 0) {
                syslog(LOG_ERR, "citserver: failed to access and create directories");
                exit(1);
        }
index 93c25674d8524af25589cbeb2eca8f22cb45f9fe..51470ddae79d8693c582ef4c8a70eac4d7e18142 100644 (file)
@@ -259,7 +259,6 @@ void CtdlModuleDoSearch(int *num_msgs, long **search_msgs, const char *search_st
 
 #define NODENAME               CtdlGetConfigStr("c_nodename")
 #define FQDN                   CtdlGetConfigStr("c_fqdn")
-#define CTDLUID                        ctdluid
 #define CREATAIDE              CtdlGetConfigInt("c_creataide")
 #define REGISCALL              CtdlGetConfigInt("c_regiscall")
 #define TWITDETECT             CtdlGetConfigInt("c_twitdetect")
index 2e7717d039e3a17576087f171197367dd549c496..8327ed50038bdc242888a3191c0cd31b20f01be7 100644 (file)
@@ -59,7 +59,7 @@ void cdb_chmod_data(void) {
                syslog(LOG_ERR, "bdb: unable to set database directory permissions [%s]: %m", ctdl_db_dir);
                exit(CTDLEXIT_DB);
        }
-       if (chown(ctdl_db_dir, CTDLUID, (-1)) != 0) {
+       if (chown(ctdl_db_dir, ctdluid, (-1)) != 0) {
                syslog(LOG_ERR, "bdb: unable to set the owner for [%s]: %m", ctdl_db_dir);
                exit(CTDLEXIT_DB);
        }
@@ -69,7 +69,7 @@ void cdb_chmod_data(void) {
                        if (d->d_name[0] != '.') {
                                snprintf(filename, sizeof filename, "%s/%s", ctdl_db_dir, d->d_name);
                                syslog(LOG_DEBUG, "bdb: chmod(%s, 0600) returned %d", filename, chmod(filename, 0600));
-                               syslog(LOG_DEBUG, "bdb: chown(%s, CTDLUID, -1) returned %d", filename, chown(filename, CTDLUID, (-1)));
+                               syslog(LOG_DEBUG, "bdb: chown(%s, ctdluid, -1) returned %d", filename, chown(filename, ctdluid, (-1)));
                        }
                }
                closedir(dp);
index 65e2bd47f744045671eea6e1576be0eca63c8a77..48fe21d9f3ccb7c661d6ea14d4a3b7d8be5da54b 100644 (file)
@@ -147,7 +147,7 @@ void cmd_setp(char *new_pw) {
        if (CtdlAccessCheck(ac_logged_in)) {
                return;
        }
-       if ( (CC->user.uid != CTDLUID) && (CC->user.uid != (-1)) ) {
+       if ( (CC->user.uid != ctdluid) && (CC->user.uid != (-1)) ) {
                cprintf("%d Not allowed.  Use the 'passwd' command.\n", ERROR + NOT_HERE);
                return;
        }
index 870205346e8156cdf3a25b0d093d666e290d5d4f..a90ddd02d748189115fe85efee7c2e45e975df88 100644 (file)
@@ -286,7 +286,7 @@ int main(int argc, char **argv) {
                cdb_chmod_data();       // make sure we own our data files
                getpwuid_r(ctdluid, &pw, pwbuf, sizeof(pwbuf), &pwp);
                if (pwp == NULL) {
-                       syslog(LOG_ERR, "main: WARNING, getpwuid(%ld): %m Group IDs will be incorrect.", (long)CTDLUID);
+                       syslog(LOG_ERR, "main: WARNING, getpwuid(%ld): %m Group IDs will be incorrect.", (long)ctdluid);
                }
                else {
                        initgroups(pw.pw_name, pw.pw_gid);
@@ -294,8 +294,8 @@ int main(int argc, char **argv) {
                                syslog(LOG_ERR, "main: setgid(%ld): %m", (long)pw.pw_gid);
                        }
                }
-               syslog(LOG_INFO, "main: changing uid to %ld", (long)CTDLUID);
-               if (setuid(CTDLUID) != 0) {
+               syslog(LOG_INFO, "main: changing uid to %ld", (long)ctdluid);
+               if (setuid(ctdluid) != 0) {
                        syslog(LOG_ERR, "main: setuid() failed: %m");
                }
 #if defined (HAVE_SYS_PRCTL_H) && defined (PR_SET_DUMPABLE)