]> code.citadel.org Git - citadel.git/blobdiff - citadel/citserver.c
* Variable names, comments, documentation, etc... removed the acronym 'BBS'
[citadel.git] / citadel / citserver.c
index 967e863bd1e6a1ed316552c5b986a6d07efbde83..67b2403805d8e7ee78ca27c96a8ff174e01ac01e 100644 (file)
@@ -68,6 +68,7 @@ char *unique_session_numbers;
 int ScheduledShutdown = 0;
 int do_defrag = 0;
 time_t server_startup_time;
+char pid_file_name[PATH_MAX];
 
 /*
  * Various things that need to be initialized at startup
@@ -77,10 +78,19 @@ void master_startup(void) {
        unsigned int seed;
        FILE *urandom;
        struct ctdlroom qrbuf;
+       FILE *pidfile_fp;
        
        lprintf(CTDL_DEBUG, "master_startup() started\n");
        time(&server_startup_time);
 
+       /* pid file.  If we go FSSTND this should end up in 'localstatedir' */
+       snprintf(pid_file_name, sizeof pid_file_name, "./citadel.pid");
+       pidfile_fp = fopen(pid_file_name, "w");
+       if (pidfile_fp != NULL) {
+               fprintf(pidfile_fp, "%d\n", (int)getpid());
+               fclose(pidfile_fp);
+       }
+
        lprintf(CTDL_INFO, "Opening databases\n");
        open_databases();
 
@@ -125,11 +135,13 @@ void master_startup(void) {
 
 /*
  * Cleanup routine to be called when the server is shutting down.
- * WARNING: It's no longer safe to call this function to force a shutdown.
- * Instead, set time_to_die = 1.
  */
-void master_cleanup(void) {
+void master_cleanup(int exitcode) {
        struct CleanupFunctionHook *fcn;
+       static int already_cleaning_up = 0;
+
+       if (already_cleaning_up) while(1) sleep(1);
+       already_cleaning_up = 1;
 
        /* Run any cleanup routines registered by loadable modules */
        for (fcn = CleanupHookTable; fcn != NULL; fcn = fcn->next) {
@@ -148,84 +160,30 @@ void master_cleanup(void) {
 #endif
 
        /* Now go away. */
-       lprintf(CTDL_NOTICE, "citserver: exiting.\n");
+       lprintf(CTDL_NOTICE, "citserver: Exiting with status %d\n", exitcode);
        fflush(stdout); fflush(stderr);
-       exit(0);
-}
-
-
-/*
- * Free any per-session data allocated by modules or whatever
- */
-void deallocate_user_data(struct CitContext *con)
-{
-       struct CtdlSessData *ptr;
 
-       begin_critical_section(S_SESSION_TABLE);
-       while (con->FirstSessData != NULL) {
-               lprintf(CTDL_DEBUG, "Deallocating user data symbol %ld\n",
-                       con->FirstSessData->sym_id);
-               if (con->FirstSessData->sym_data != NULL)
-                       free(con->FirstSessData->sym_data);
-               ptr = con->FirstSessData->next;
-               free(con->FirstSessData);
-               con->FirstSessData = ptr;
-       }
-       end_critical_section(S_SESSION_TABLE);
+       unlink(pid_file_name);
+       exit(exitcode);
 }
 
 
 
-
 /*
- * Terminate a session and remove its context data structure.
+ * Terminate a session.
  */
 void RemoveContext (struct CitContext *con)
 {
-       struct CitContext *ptr = NULL;
-       struct CitContext *ToFree = NULL;
-
        if (con==NULL) {
-               lprintf(CTDL_ERR, "WARNING: RemoveContext() called with NULL!\n");
-               return;
-       }
-       lprintf(CTDL_DEBUG, "RemoveContext() called\n");
-
-       /* Remove the context from the global context list.  This needs
-        * to get done FIRST to avoid concurrency problems.  It is *vitally*
-        * important to keep num_sessions accurate!!
-        */
-       lprintf(CTDL_DEBUG, "Removing context for session %d\n", con->cs_pid);
-       begin_critical_section(S_SESSION_TABLE);
-       if (ContextList == con) {
-               ToFree = ContextList;
-               ContextList = ContextList->next;
-               --num_sessions;
-       }
-       else {
-               for (ptr = ContextList; ptr != NULL; ptr = ptr->next) {
-                       if (ptr->next == con) {
-                               /* See fair scheduling in sysdep.c */
-                               if (next_session == ptr->next)
-                                       next_session = ptr->next->next;
-                               ToFree = ptr->next;
-                               ptr->next = ptr->next->next;
-                               --num_sessions;
-                       }
-               }
-       }
-       end_critical_section(S_SESSION_TABLE);
-
-       if (ToFree == NULL) {
-               lprintf(CTDL_DEBUG, "RemoveContext() found nothing to remove\n");
+               lprintf(CTDL_ERR,
+                       "WARNING: RemoveContext() called with NULL!\n");
                return;
        }
+       lprintf(CTDL_DEBUG, "RemoveContext() session %d\n", con->cs_pid);
 
        /* Run any cleanup routines registered by loadable modules.
-        * Note 1: This must occur *before* deallocate_user_data() because the
-        *         cleanup functions might touch dynamic session data.
-        * Note 2: We have to "become_session()" because the cleanup functions
-        *         might make references to "CC" assuming it's the right one.
+        * Note: We have to "become_session()" because the cleanup functions
+        *       might make references to "CC" assuming it's the right one.
         */
        become_session(con);
        PerformSessionHooks(EVT_STOP);
@@ -238,18 +196,10 @@ void RemoveContext (struct CitContext *con)
        unlink(con->temp);
        lprintf(CTDL_NOTICE, "[%3d] Session ended.\n", con->cs_pid);
 
-       /* Deallocate any user-data attached to this session */
-       deallocate_user_data(con);
-
        /* If the client is still connected, blow 'em away. */
        lprintf(CTDL_DEBUG, "Closing socket %d\n", con->client_socket);
        close(con->client_socket);
 
-       /* This is where we used to check for scheduled shutdowns. */
-
-       /* Free up the memory used by this context */
-       free(con);
-
        lprintf(CTDL_DEBUG, "Done with RemoveContext()\n");
 }
 
@@ -257,81 +207,6 @@ void RemoveContext (struct CitContext *con)
 
 
 
-/*
- * Return a pointer to some generic per-session user data.
- * (This function returns NULL if the requested symbol is not allocated.)
- *
- * NOTE: we use critical sections for allocating and de-allocating these,
- *       but not for locating one.
- */
-void *CtdlGetUserData(unsigned long requested_sym) 
-{
-       struct CtdlSessData *ptr;
-
-       for (ptr = CC->FirstSessData; ptr != NULL; ptr = ptr->next)
-               if (ptr->sym_id == requested_sym)
-                       return(ptr->sym_data);
-
-       lprintf(CTDL_ERR, "ERROR! CtdlGetUserData(%ld) symbol not allocated\n",
-               requested_sym);
-       return NULL;
-}
-
-
-/*
- * Allocate some generic per-session user data.
- */
-void CtdlAllocUserData(unsigned long requested_sym, size_t num_bytes)
-{
-       struct CtdlSessData *ptr;
-
-       lprintf(CTDL_DEBUG, "CtdlAllocUserData(%ld) called\n", requested_sym);
-
-       /* Fail silently if the symbol is already registered. */
-       for (ptr = CC->FirstSessData; ptr != NULL; ptr = ptr->next)  {
-               if (ptr->sym_id == requested_sym) {
-                       return;
-               }
-       }
-
-       /* Grab us some memory!  Dem's good eatin' !!  */
-       ptr = malloc(sizeof(struct CtdlSessData));
-       ptr->sym_id = requested_sym;
-       ptr->sym_data = malloc(num_bytes);
-       memset(ptr->sym_data, 0, num_bytes);
-
-       begin_critical_section(S_SESSION_TABLE);
-       ptr->next = CC->FirstSessData;
-       CC->FirstSessData = ptr;
-       end_critical_section(S_SESSION_TABLE);
-
-       lprintf(CTDL_DEBUG, "CtdlAllocUserData(%ld) finished\n", requested_sym);
-}
-
-
-/* 
- * Change the size of a buffer allocated with CtdlAllocUserData()
- */
-void CtdlReallocUserData(unsigned long requested_sym, size_t num_bytes)
-{
-       struct CtdlSessData *ptr;
-
-       for (ptr = CC->FirstSessData; ptr != NULL; ptr = ptr->next)  {
-               if (ptr->sym_id == requested_sym) {
-                       ptr->sym_data = realloc(ptr->sym_data, num_bytes);
-                       return;
-               }
-       }
-
-       lprintf(CTDL_ERR, "CtdlReallocUserData() ERROR: symbol %ld not found!\n",
-               requested_sym);
-}
-
-
-
-
-
-
 /*
  * cmd_info()  -  tell the client about this server
  */
@@ -343,7 +218,7 @@ void cmd_info(void) {
        cprintf("%s\n", config.c_fqdn);
        cprintf("%s\n", CITADEL);
        cprintf("%d\n", REV_LEVEL);
-       cprintf("%s\n", config.c_bbs_city);
+       cprintf("%s\n", config.c_site_location);
        cprintf("%s\n", config.c_sysadm);
        cprintf("%d\n", SERVER_TYPE);
        cprintf("%s\n", config.c_moreprompt);
@@ -398,8 +273,8 @@ void cmd_time(void)
  */
 int is_public_client(void)
 {
-       char buf[SIZ];
-       char addrbuf[SIZ];
+       char buf[1024];
+       char addrbuf[1024];
        FILE *fp;
        int i;
        struct stat statbuf;
@@ -424,7 +299,7 @@ int is_public_client(void)
                begin_critical_section(S_PUBLIC_CLIENTS);
                lprintf(CTDL_INFO, "Loading %s\n", PUBLIC_CLIENTS);
 
-               strcpy(public_clients, "127.0.0.1");
+               safestrncpy(public_clients, "127.0.0.1", sizeof public_clients);
                if (hostname_to_dotted_quad(addrbuf, config.c_fqdn) == 0) {
                        strcat(public_clients, "|");
                        strcat(public_clients, addrbuf);
@@ -454,7 +329,7 @@ int is_public_client(void)
        lprintf(CTDL_DEBUG, "Checking whether %s is a local or public client\n",
                CC->cs_addr);
        for (i=0; i<num_parms(public_clients); ++i) {
-               extract(addrbuf, public_clients, i);
+               extract_token(addrbuf, public_clients, i, '|', sizeof addrbuf);
                if (!strcasecmp(CC->cs_addr, addrbuf)) {
                        lprintf(CTDL_DEBUG, "... yes it is.\n");
                        return(1);
@@ -475,8 +350,8 @@ void cmd_iden(char *argbuf)
        int dev_code;
        int cli_code;
        int rev_level;
-       char desc[SIZ];
-       char from_host[SIZ];
+       char desc[128];
+       char from_host[128];
        struct in_addr addr;
        int do_lookup = 0;
 
@@ -488,11 +363,11 @@ void cmd_iden(char *argbuf)
        dev_code = extract_int(argbuf,0);
        cli_code = extract_int(argbuf,1);
        rev_level = extract_int(argbuf,2);
-       extract(desc,argbuf,3);
+       extract_token(desc, argbuf, 3, '|', sizeof desc);
 
        safestrncpy(from_host, config.c_fqdn, sizeof from_host);
        from_host[sizeof from_host - 1] = 0;
-       if (num_parms(argbuf)>=5) extract(from_host,argbuf,4);
+       if (num_parms(argbuf)>=5) extract_token(from_host, argbuf, 4, '|', sizeof from_host);
 
        CC->cs_clientdev = dev_code;
        CC->cs_clienttyp = cli_code;
@@ -535,24 +410,22 @@ void cmd_iden(char *argbuf)
 void cmd_mesg(char *mname)
 {
        FILE *mfp;
-       char targ[SIZ];
-       char buf[SIZ];
-       char buf2[SIZ];
+       char targ[256];
+       char buf[256];
+       char buf2[256];
        char *dirs[2];
 
-       extract(buf,mname,0);
+       extract_token(buf, mname, 0, '|', sizeof buf);
 
-       dirs[0]=malloc(64);
-       dirs[1]=malloc(64);
-       strcpy(dirs[0],"messages");
-       strcpy(dirs[1],"help");
+       dirs[0] = strdup("messages");
+       dirs[1] = strdup("help");
        snprintf(buf2, sizeof buf2, "%s.%d.%d", buf, CC->cs_clientdev, CC->cs_clienttyp);
-       mesg_locate(targ,sizeof targ,buf2,2,(const char **)dirs);
+       mesg_locate(targ, sizeof targ, buf2, 2, (const char **)dirs);
        if (strlen(targ) == 0) {
                snprintf(buf2, sizeof buf2, "%s.%d", buf, CC->cs_clientdev);
-               mesg_locate(targ,sizeof targ,buf2,2,(const char **)dirs);
+               mesg_locate(targ, sizeof targ, buf2, 2, (const char **)dirs);
                if (strlen(targ) == 0) {
-                       mesg_locate(targ,sizeof targ,buf,2,(const char **)dirs);
+                       mesg_locate(targ, sizeof targ, buf, 2, (const char **)dirs);
                }       
        }
        free(dirs[0]);
@@ -571,7 +444,7 @@ void cmd_mesg(char *mname)
        }
        cprintf("%d %s\n",LISTING_FOLLOWS,buf);
 
-       while (fgets(buf, (SIZ-1), mfp)!=NULL) {
+       while (fgets(buf, (sizeof buf - 1), mfp) != NULL) {
                buf[strlen(buf)-1] = 0;
                do_help_subst(buf);
                cprintf("%s\n",buf);
@@ -588,8 +461,8 @@ void cmd_mesg(char *mname)
 void cmd_emsg(char *mname)
 {
        FILE *mfp;
-       char targ[SIZ];
-       char buf[SIZ];
+       char targ[256];
+       char buf[256];
        char *dirs[2];
        int a;
 
@@ -597,16 +470,14 @@ void cmd_emsg(char *mname)
 
        if (CtdlAccessCheck(ac_aide)) return;
 
-       extract(buf,mname,0);
+       extract_token(buf, mname, 0, '|', sizeof buf);
        for (a=0; a<strlen(buf); ++a) {         /* security measure */
                if (buf[a] == '/') buf[a] = '.';
        }
 
-       dirs[0]=malloc(64);
-       dirs[1]=malloc(64);
-       strcpy(dirs[0],"messages");
-       strcpy(dirs[1],"help");
-       mesg_locate(targ,sizeof targ,buf,2,(const char**)dirs);
+       dirs[0] = strdup("messages");
+       dirs[1] = strdup("help");
+       mesg_locate(targ, sizeof targ, buf, 2, (const char**)dirs);
        free(dirs[0]);
        free(dirs[1]);
 
@@ -622,7 +493,7 @@ void cmd_emsg(char *mname)
        }
        cprintf("%d %s\n", SEND_LISTING, targ);
 
-       while (client_gets(buf), strcmp(buf, "000")) {
+       while (client_getln(buf, sizeof buf), strcmp(buf, "000")) {
                fprintf(mfp, "%s\n", buf);
        }
 
@@ -637,13 +508,15 @@ void GenerateRoomDisplay(char *real_room,
                        struct CitContext *viewed,
                        struct CitContext *viewer) {
 
+       int ra;
+
        strcpy(real_room, viewed->room.QRname);
        if (viewed->room.QRflags & QR_MAILBOX) {
                strcpy(real_room, &real_room[11]);
        }
        if (viewed->room.QRflags & QR_PRIVATE) {
-               if ( (CtdlRoomAccess(&viewed->room, &viewer->user)
-                  & UA_KNOWN) == 0) {
+               CtdlRoomAccess(&viewed->room, &viewer->user, &ra, NULL);
+               if ( (ra & UA_KNOWN) == 0) {
                        strcpy(real_room, "<private room>");
                }
        }
@@ -776,7 +649,8 @@ void cmd_ipgm(char *argbuf)
         */
        if (!CC->is_local_socket) {
                sleep(5);
-               cprintf("%d Authentication failed.\n", ERROR + PASSWORD_REQUIRED);
+               cprintf("%d Authentication failed.\n",
+                       ERROR + PASSWORD_REQUIRED);
        }
        else if (secret == config.c_ipgm_secret) {
                CC->internal_pgm = 1;
@@ -786,15 +660,21 @@ void cmd_ipgm(char *argbuf)
        }
        else {
                sleep(5);
-               cprintf("%d Authentication failed.\n", ERROR + PASSWORD_REQUIRED);
+               cprintf("%d Authentication failed.\n",
+                       ERROR + PASSWORD_REQUIRED);
                lprintf(CTDL_ERR, "Warning: ipgm authentication failed.\n");
                CC->kill_me = 1;
        }
 
-       /* Now change the ipgm secret for the next round. */
+       /* Now change the ipgm secret for the next round.
+        * (Disabled because it breaks concurrent scripts.  The fact that
+        * we no longer accept IPGM over the network should be sufficient
+        * to prevent brute-force attacks.  If you don't agree, uncomment
+        * this block.)
        get_config();
        config.c_ipgm_secret = rand();
        put_config();
+       */
 }
 
 
@@ -905,7 +785,6 @@ void begin_session(struct CitContext *con)
        con->cs_flags = 0;
        con->upload_type = UPL_FILE;
        con->dl_is_net = 0;
-       con->FirstSessData = NULL;
 
        con->nologin = 0;
        if ((config.c_maxsessions > 0)&&(num_sessions > config.c_maxsessions))
@@ -938,16 +817,24 @@ void citproto_begin_session() {
  * This loop recognizes all server commands.
  */
 void do_command_loop(void) {
-       char cmdbuf[SIZ];
+       char cmdbuf[1024];
 
        time(&CC->lastcmd);
        memset(cmdbuf, 0, sizeof cmdbuf); /* Clear it, just in case */
-       if (client_gets(cmdbuf) < 1) {
+       if (client_getln(cmdbuf, sizeof cmdbuf) < 1) {
                lprintf(CTDL_ERR, "Client socket is broken; ending session\n");
                CC->kill_me = 1;
                return;
        }
-       lprintf(CTDL_INFO, "Citadel: %s\n", cmdbuf);
+
+       /* Log the server command, but don't show passwords... */
+       if ( (strncasecmp(cmdbuf, "PASS", 4))
+          && (strncasecmp(cmdbuf, "SETP", 4)) ) {
+               lprintf(CTDL_INFO, "%s\n", cmdbuf);
+       }
+       else {
+               lprintf(CTDL_INFO, "<password command sent>\n");
+       }
 
        buffer_output();
 
@@ -960,8 +847,7 @@ void do_command_loop(void) {
           && (strncasecmp(cmdbuf, "PEXP", 4))
           && (strncasecmp(cmdbuf, "GEXP", 4)) ) {
                strcpy(CC->lastcmdname, "    ");
-               safestrncpy(CC->lastcmdname, cmdbuf, 
-                       sizeof(CC->lastcmdname) );
+               safestrncpy(CC->lastcmdname, cmdbuf, sizeof(CC->lastcmdname));
                time(&CC->lastidle);
        }
                
@@ -972,11 +858,11 @@ void do_command_loop(void) {
           CC->cs_flags &= ~CS_POSTING;
        }
                   
-       if (!strncasecmp(cmdbuf,"NOOP",4)) {
+       if (!strncasecmp(cmdbuf, "NOOP", 4)) {
                cprintf("%d%cok\n", CIT_OK, CtdlCheckExpress() );
        }
        
-       else if (!strncasecmp(cmdbuf,"QNOP",4)) {
+       else if (!strncasecmp(cmdbuf, "QNOP", 4)) {
                /* do nothing, this command returns no response */
        }