From: Lyndon Nerenberg <lyndon@orthanc.ca>
Date: Mon, 1 Jul 2013 00:59:31 +0000 (-0700)
Subject: Move a variable manipulation outside the context of setjmp() to quell a compiler... 
X-Git-Url: https://diplodocus.org/git/nmh/commitdiff_plain/0152658e840581077e5cd575b5195faf18b9df8b?ds=inline;hp=--cc

Move a variable manipulation outside the context of setjmp() to quell a compiler warning.
---

0152658e840581077e5cd575b5195faf18b9df8b
diff --git a/uip/mhlsbr.c b/uip/mhlsbr.c
index 82e65d77..e94e8d3e 100644
--- a/uip/mhlsbr.c
+++ b/uip/mhlsbr.c
@@ -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;
diff --git a/uip/rcvtty.c b/uip/rcvtty.c
index 6282d522..85b9a484 100644
--- a/uip/rcvtty.c
+++ b/uip/rcvtty.c
@@ -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 */