]> 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 c2297a46920dce2608852fd3261bbca093c4c3fe..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();
 
@@ -153,60 +163,27 @@ void master_cleanup(int exitcode) {
        lprintf(CTDL_NOTICE, "citserver: Exiting with status %d\n", exitcode);
        fflush(stdout); fflush(stderr);
 
+       unlink(pid_file_name);
        exit(exitcode);
 }
 
 
-/*
- * 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);
-}
-
-
-
 
 /*
- * Terminate a session and remove its context data structure.
+ * Terminate a session.
  */
 void RemoveContext (struct CitContext *con)
 {
        if (con==NULL) {
-               lprintf(CTDL_ERR, "WARNING: RemoveContext() called with NULL!\n");
+               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 (con->prev) con->prev->next = con->next; else ContextList = con->next;
-         if (con->next) con->next->prev = con->prev;
-         --num_sessions;
-       end_critical_section(S_SESSION_TABLE);
+       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);
@@ -219,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");
 }
 
@@ -238,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
  */
@@ -324,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);
@@ -379,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;
@@ -405,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);
@@ -435,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);
@@ -456,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;
 
@@ -469,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;
@@ -516,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]);
@@ -552,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);
@@ -569,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;
 
@@ -578,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]);
 
@@ -759,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;
@@ -769,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();
+       */
 }
 
 
@@ -888,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))
@@ -921,7 +817,7 @@ 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 */
@@ -951,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);
        }
                
@@ -963,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 */
        }