]> code.citadel.org Git - citadel.git/blobdiff - citadel/citserver.c
More session table stability nonsense
[citadel.git] / citadel / citserver.c
index e625d761cc4f1dcb988b5087d3c01d3a2056131c..15c6d0c6c1239c439e51b32146844a796d317796 100644 (file)
@@ -1,3 +1,4 @@
+/* $Id$ */
 #include <stdlib.h>
 #include <unistd.h>
 #include <stdio.h>
@@ -7,6 +8,7 @@
 #include <ctype.h>
 #include <string.h>
 #include <errno.h>
+#include <limits.h>
 #include <pthread.h>
 #include <syslog.h>
 #include <dlfcn.h>
@@ -17,7 +19,6 @@
 #include "config.h"
 #include "database.h"
 #include "housekeeping.h"
-#include "hooks.h"
 #include "user_ops.h"
 #include "logging.h"
 #include "support.h"
@@ -26,6 +27,9 @@
 #include "room_ops.h"
 #include "file_ops.h"
 #include "dynloader.h"
+#include "policy.h"
+#include "control.h"
+#include "tools.h"
 
 struct CitContext *ContextList = NULL;
 int ScheduledShutdown = 0;
@@ -39,13 +43,18 @@ void master_startup(void) {
 
        lprintf(7, "Checking floor reference counts\n");
        check_ref_counts();
-       
+
+       lprintf(7, "Creating base rooms (if necessary)\n");
+       create_room(BASEROOM, 0, "", 0);
+       create_room(AIDEROOM, 4, "", 0);
+       create_room(config.c_twitroom, 0, "", 0);
        }
 
 /*
  * Cleanup routine to be called when the server is shutting down.
  */
 void master_cleanup(void) {
+       struct CleanupFunctionHook *fcn;
 
        /* Cancel all running sessions */
        lprintf(7, "Cancelling running sessions...\n");
@@ -53,6 +62,11 @@ void master_cleanup(void) {
                kill_session(ContextList->cs_pid);
                }
 
+       /* Run any cleanup routines registered by loadable modules */
+       for (fcn = CleanupHookTable; fcn != NULL; fcn = fcn->next) {
+               (*fcn->h_function_pointer)();
+               }
+
        /* Close databases */
        lprintf(7, "Closing databases\n");
        close_databases();
@@ -61,8 +75,8 @@ void master_cleanup(void) {
        sysdep_master_cleanup();
 
        /* Now go away. */
-       hook_cleanup();
        lprintf(3, "citserver: exiting.\n");
+       fflush(stdout); fflush(stderr);
        exit(0);
        }
 
@@ -86,7 +100,10 @@ void cleanup_stuff(void *arg)
        rec_log(CL_TERMINATE,CC->curr_user);
        unlink(CC->temp);
        lprintf(3, "citserver[%3d]: ended.\n",CC->cs_pid);
-       hook_end_session(CC->cs_pid);
+       
+       /* Run any cleanup routines registered by loadable modules */
+       PerformSessionHooks(EVT_STOP);
+
        syslog(LOG_NOTICE,"session %d ended", CC->cs_pid);
        
        /* Deallocate any unsent express messages */
@@ -102,7 +119,7 @@ void cleanup_stuff(void *arg)
        if (CC->msglist != NULL) free(CC->msglist);
 
        /* Now get rid of the session and context */
-       lprintf(7, "cleanup_stuff() is calling RemoveContext(%d)\n", CC->cs_pid);
+       lprintf(7, "cleanup_stuff() calling RemoveContext(%d)\n", CC->cs_pid);
        RemoveContext(CC);
 
        /* While we still have an extra thread with no user attached to it,
@@ -117,13 +134,32 @@ void cleanup_stuff(void *arg)
  */
 void set_wtmpsupp(char *newtext)
 {
-       strncpy(CC->cs_room,newtext,19);
-       CC->cs_room[19] = 0;
+       strncpy(CC->cs_room,newtext,sizeof CC->cs_room);
+       CC->cs_room[sizeof CC->cs_room - 1] = 0;
        time(&CC->cs_lastupdt);
-       hook_room_name(CC->cs_pid, CC->cs_room);
+
+       /* Run any routines registered by loadable modules */
+       PerformSessionHooks(EVT_NEWROOM);
        }
 
 
+/*
+ * call set_wtmpsupp() with the name of the current room, modified a bit...
+ */
+void set_wtmpsupp_to_current_room() {
+       if (CC->quickroom.QRflags & QR_PRIVATE) {
+               set_wtmpsupp("<private room>");
+               }
+       else if (CC->quickroom.QRflags & QR_MAILBOX) {
+               set_wtmpsupp(&CC->quickroom.QRname[11]);
+               }
+       else {
+               set_wtmpsupp(CC->quickroom.QRname);
+               }
+       }
+
+
+
 /*
  * cmd_info()  -  tell the client about this server
  */
@@ -143,16 +179,19 @@ void cmd_info(void) {
        cprintf("000\n");
        }
 
-void cmd_rchg(char *newroomname)
+void cmd_rchg(char *argbuf)
 {
-   if ((newroomname) && (newroomname[0]))
-   {
-      bzero(CC->fake_roomname, 20);
-      strncpy(CC->fake_roomname, newroomname, 19);
-   }
-   else
-      CC->fake_roomname[0] = '\0';
-   cprintf("%d\n",OK);
+       char newroomname[ROOMNAMELEN];
+
+       extract(newroomname, argbuf, 0);
+       if (strlen(newroomname) > 0) {
+               strncpy(CC->fake_roomname, newroomname, ROOMNAMELEN);
+               CC->fake_roomname[ROOMNAMELEN - 1] = 0;
+               }
+       else {
+               strcpy(CC->fake_roomname, "");
+               }
+       cprintf("%d OK\n", OK);
 }
 
 void cmd_hchg(char *newhostname)
@@ -163,8 +202,8 @@ void cmd_hchg(char *newhostname)
       strncpy(CC->fake_hostname, newhostname, 24);
    }
    else
-      CC->fake_hostname[0] = '\0';
-   cprintf("%d\n",OK);
+      strcpy(CC->fake_hostname, "");
+   cprintf("%d OK\n",OK);
 }
 
 void cmd_uchg(char *newusername)
@@ -261,7 +300,8 @@ void cmd_iden(char *argbuf)
        rev_level = extract_int(argbuf,2);
        extract(desc,argbuf,3);
 
-       strcpy(from_host,config.c_fqdn);
+       strncpy(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);
 
        CC->cs_clientdev = dev_code;
@@ -275,7 +315,7 @@ void cmd_iden(char *argbuf)
                strncpy(CC->cs_host,from_host,24);
                CC->cs_host[24] = 0;
                }
-       set_wtmpsupp(CC->quickroom.QRname);
+       set_wtmpsupp_to_current_room();
 
        syslog(LOG_NOTICE,"client %d/%d/%01d.%02d (%s)\n",
                dev_code,
@@ -317,7 +357,7 @@ void cmd_stel(char *cmdbuf)
                        CC->cs_flags = CC->cs_flags|CS_STEALTH;
                }
 
-       set_wtmpsupp(CC->quickroom.QRname);
+       set_wtmpsupp_to_current_room();
        cprintf("%d Ok\n",OK);
        }
 
@@ -401,7 +441,7 @@ void cmd_emsg(char *mname)
        free(dirs[1]);
 
        if (strlen(targ)==0) {
-               sprintf(targ, "./help/%s", buf);
+               snprintf(targ, sizeof targ, "./help/%s", buf);
                }
 
        mfp = fopen(targ,"w");
@@ -502,6 +542,7 @@ void cmd_term(char *cmdbuf)
 {
        int session_num;
        struct CitContext *ccptr;
+       int session_to_kill = 0;
 
        if (!CC->logged_in) {
                cprintf("%d Not logged in.\n",ERROR+NOT_LOGGED_IN);
@@ -520,15 +561,23 @@ void cmd_term(char *cmdbuf)
                return;
                }
 
+       lprintf(9, "Locating session to kill\n");
+       begin_critical_section(S_SESSION_TABLE);
        for (ccptr = ContextList; ccptr != NULL; ccptr = ccptr->next) {
                if (session_num == ccptr->cs_pid) {
-                       kill_session(ccptr->cs_pid);
-                       cprintf("%d Session terminated.\n", OK);
-                       return;
+                       session_to_kill = ccptr->cs_pid;
                        }
                }
+       end_critical_section(S_SESSION_TABLE);
+       lprintf(9, "session_to_kill == %d\n", session_to_kill);
 
-       cprintf("%d No such session.\n", ERROR);
+       if (session_to_kill > 0) {
+               kill_session(ccptr->cs_pid);
+               cprintf("%d Session terminated.\n", OK);
+               }
+       else {
+               cprintf("%d No such session.\n", ERROR);
+               }
        }
 
 
@@ -617,68 +666,13 @@ void cmd_scdn(char *argbuf)
        }
 
 
-/*
- * Run a server extension (FIX FIX FIX initial hack; polish this up)
- */
-void cmd_extn(char *argbuf) {
-       char ExtensionName[256];
-       int is_ipgm;
-       int pid;
-       char portstr[16];
-       char ipgm[32];
-       char sess[16];
-
-
-       extract(ExtensionName, argbuf, 0);
-       is_ipgm = extract_int(argbuf, 1);
-       
-       pid = fork();
-       lprintf(9, "fork() returned %d\n", pid);
-       if (pid < 0) {
-               cprintf("%d fork failed: %s\n",
-                       ERROR + INTERNAL_ERROR,
-                       strerror(errno));
-               return;
-               }
-       else if (pid == 0) {
-
-               sprintf(portstr, "%d", config.c_port_number);
-
-               if (is_ipgm)
-                       sprintf(ipgm, "%d", config.c_ipgm_secret);
-               else
-                       strcpy(ipgm, "");
-
-               sprintf(sess, "%d", CC->cs_pid);
-
-               execlp(ExtensionName, ExtensionName,
-                       "localhost",                    /* server address */
-                       portstr,                        /* port number */
-                       ipgm,                           /* ipgm secret */
-                       CC->usersupp.fullname,          /* user name */
-                       CC->usersupp.password,          /* password */
-                       CC->quickroom.QRname,           /* current room */
-                       sess,                           /* assoc session id */
-                       NULL);
-       
-               lprintf(9, "exec() failed: %s\n", strerror(errno));
-               exit(1);
-               }
-       else {
-               cprintf("%d Ok\n", OK);
-               }
-       }
-
-
-
-
 /*
  * main context loop
  */
 void *context_loop(struct CitContext *con)
 {
        char cmdbuf[256];
-       int session_num;
+       int num_sessions;
 
        /*
         * Wedge our way into the context table.
@@ -688,7 +682,6 @@ void *context_loop(struct CitContext *con)
        /* 
         * Initialize some variables specific to our context.
         */
-       CC->curr_rm = (-1);
        CC->logged_in = 0;
        CC->internal_pgm = 0;
        CC->download_fp = NULL;
@@ -703,17 +696,18 @@ void *context_loop(struct CitContext *con)
        strcpy(CC->cs_clientname, "(unknown)");
        strcpy(CC->curr_user,"");
        strcpy(CC->net_node,"");
-       sprintf(CC->temp,"/tmp/CitServer.%d.%d", getpid(), CC->cs_pid);
+       snprintf(CC->temp, sizeof CC->temp, "/tmp/CitServer.%d.%d", getpid(), CC->cs_pid);
        strcpy(CC->cs_room, "");
-       strcpy(CC->cs_host, config.c_fqdn);
+       strncpy(CC->cs_host, config.c_fqdn, sizeof CC->cs_host);
+       CC->cs_host[sizeof CC->cs_host - 1] = 0;
        locate_host(CC->cs_host);
        CC->cs_flags = 0;
        CC->upload_type = UPL_FILE;
        CC->dl_is_net = 0;
 
-       session_num = session_count();
+       num_sessions = session_count();
        CC->nologin = 0;
-       if ((config.c_maxsessions > 0)&&(session_num >= config.c_maxsessions))
+       if ((config.c_maxsessions > 0)&&(num_sessions > config.c_maxsessions))
                CC->nologin = 1;
 
        if (CC->nologin==1) {
@@ -726,14 +720,17 @@ void *context_loop(struct CitContext *con)
                }
 
        lprintf(3, "citserver[%3d]: started.\n", CC->cs_pid);
-       hook_start_session(CC->cs_pid);
+
+       /* Run any session startup routines registered by loadable modules */
+       PerformSessionHooks(EVT_START);
+
        rec_log(CL_CONNECT, "");
 
        do {
                time(&CC->lastcmd);
+               bzero(cmdbuf, sizeof cmdbuf); /* Clear it, just in case */
                if (client_gets(cmdbuf) < 1) cleanup(EXIT_NULL);
                lprintf(5, "citserver[%3d]: %s\n", CC->cs_pid, cmdbuf);
-               hook_command_received(CC->cs_pid, cmdbuf);
 
                /*
                 * Let other clients see the last command we executed, but
@@ -1083,10 +1080,6 @@ void *context_loop(struct CitContext *con)
                        cmd_rchg(&cmdbuf[5]);
                        }
 
-               else if (!strncasecmp(cmdbuf, "EXTN", 4)) {
-                       cmd_extn(&cmdbuf[5]);
-                       }
-
                else if (!strncasecmp(cmdbuf, "AGUP", 4)) {
                        cmd_agup(&cmdbuf[5]);
                        }
@@ -1095,6 +1088,18 @@ void *context_loop(struct CitContext *con)
                        cmd_asup(&cmdbuf[5]);
                        }
 
+               else if (!strncasecmp(cmdbuf, "GPEX", 4)) {
+                       cmd_gpex(&cmdbuf[5]);
+                       }
+
+               else if (!strncasecmp(cmdbuf, "SPEX", 4)) {
+                       cmd_spex(&cmdbuf[5]);
+                       }
+
+               else if (!strncasecmp(cmdbuf, "CONF", 4)) {
+                       cmd_conf(&cmdbuf[5]);
+                       }
+
                else if (!DLoader_Exec_Cmd(cmdbuf))
                        {
                           cprintf("%d Unrecognized or unsupported command.\n",