]> diplodocus.org Git - nmh/commitdiff
Move a variable manipulation outside the context of setjmp() to quell a compiler...
authorLyndon Nerenberg <lyndon@orthanc.ca>
Mon, 1 Jul 2013 00:59:31 +0000 (17:59 -0700)
committerLyndon Nerenberg <lyndon@orthanc.ca>
Mon, 1 Jul 2013 00:59:31 +0000 (17:59 -0700)
uip/mhlsbr.c
uip/rcvtty.c

index 82e65d77f26301ee5bca976aba791df49e23017e..e94e8d3e84d70ae2b34349136499369f458edb91 100644 (file)
@@ -870,9 +870,10 @@ parse (void)
  */
 
 static void
-process (char *folder, char *fname, int ofilen, int ofilec)
+process (char *folder, char *filename, int ofilen, int ofilec)
 {
     char *cp = NULL;
+    char *fname = filename;
     FILE *fp = NULL;
     struct mcomp *c1;
     struct stat st;
index 6282d52233fdf54bd62be8ec153d0ce659cc3214..85b9a4845a9a7ca5573537971105a45c1b0748f8 100644 (file)
@@ -183,48 +183,72 @@ static int
 message_fd (char **vec)
 {
     pid_t child_id;
-    int bytes, fd, seconds;
+    int bytes, fd, seconds, bailout;
     char tmpfil[BUFSIZ];
     struct stat st;
 
+    bailout = 0;
     fd = mkstemp (strncpy (tmpfil, "/tmp/rcvttyXXXXX", sizeof(tmpfil)));
     unlink (tmpfil);
 
     if ((child_id = fork()) == NOTOK) {
+
        /* fork error */
        close (fd);
        return header_fd ();
+
     } else if (child_id) {
+
        /* parent process */
        if (!setjmp (myctx)) {
+
            SIGNAL (SIGALRM, alrmser);
            bytes = fstat(fileno (stdin), &st) != NOTOK ? (int) st.st_size : 100;
 
            /* amount of time to wait depends on message size */
            if (bytes <= 100) {
+
                /* give at least 5 minutes */
                seconds = 300;
+
            } else if (bytes >= 90000) {
+
                /* but 30 minutes should be long enough */
                seconds = 1800;
+
            } else {
+
                seconds = (bytes / 60) + 300;
+
            }
+
            alarm ((unsigned int) seconds);
            pidwait(child_id, OK);
            alarm (0);
 
-           if (fstat (fd, &st) != NOTOK && st.st_size > (off_t) 0)
-               return fd;
        } else {
+
            /*
             * Ruthlessly kill the child and anything
             * else in its process group.
             */
            killpg(child_id, SIGKILL);
+           bailout = 1;
+
+       }
+
+       if (bailout) {
+
+            if (fstat (fd, &st) != NOTOK && st.st_size > (off_t) 0) {
+               return fd;
+           }
+
+       } else {
+
+           close (fd);
+           return header_fd ();
+
        }
-       close (fd);
-       return header_fd ();
     }
 
     /* child process */