From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from localhost (fantadrom.bsd.lv [local]);
	by fantadrom.bsd.lv (OpenSMTPD) with ESMTPA id 6144f9a1;
	for <source@mdocml.bsd.lv>;
	Mon, 9 Mar 2015 22:02:58 -0500 (EST)
Date: Mon, 9 Mar 2015 22:02:58 -0500 (EST)
Message-Id: <7968710172954842175.enqueue@fantadrom.bsd.lv>
X-Mailinglist: mdocml-source
Reply-To: source@mdocml.bsd.lv
MIME-Version: 1.0
From: schwarze@mdocml.bsd.lv
To: source@mdocml.bsd.lv
Subject: mdocml: Fix a regression caused in rev.
X-Mailer: activitymail 1.26, http://search.cpan.org/dist/activitymail/
Content-Type: text/plain; charset=utf-8

Log Message:
-----------
Fix a regression caused in rev. 1.212, reported by kristaps@:

When using a pager and the first manual shown is gzip'ed,
the gunzip(1) process ended up as a child of the pager process
such that the man(1) process couldn't wait for it, preventing
proper display of the manual.

Solve this by making the pager a child of the man(1) process
(instead of the other way round), which requires being a bit
more careful about properly closing file descriptors after use
and waiting for the pager before exiting man(1).

Modified Files:
--------------
    mdocml:
        main.c

Revision Data
-------------
Index: main.c
===================================================================
RCS file: /home/cvs/mdocml/mdocml/main.c,v
retrieving revision 1.223
retrieving revision 1.224
diff -Lmain.c -Lmain.c -u -p -r1.223 -r1.224
--- main.c
+++ main.c
@@ -20,6 +20,7 @@
 
 #include <sys/types.h>
 #include <sys/param.h>	/* MACHINE */
+#include <sys/wait.h>
 
 #include <assert.h>
 #include <ctype.h>
@@ -481,6 +482,21 @@ out:
 
 	free(defos);
 
+	/*
+	 * Flush the output and signal end of file.
+	 * If a pager is attached, it allows browsing to the end.
+	 * Otherwise, it does no harm, we are about to exit anyway.
+	 */
+
+	fclose(stdout);
+
+	/*
+	 * If we spawned a pager, wait for the user to close it.
+	 * Otherwise, this call fails with no adverse effect.
+	 */
+
+	wait(NULL);
+
 	return((int)rc);
 }
 
@@ -951,18 +967,19 @@ spawn_pager(void)
 		    progname, strerror(errno));
 		exit((int)MANDOCLEVEL_SYSERR);
 	case 0:
+		break;
+	default:
 		close(fildes[0]);
 		if (dup2(fildes[1], STDOUT_FILENO) == -1) {
 			fprintf(stderr, "%s: dup output: %s\n",
 			    progname, strerror(errno));
 			exit((int)MANDOCLEVEL_SYSERR);
 		}
+		close(fildes[1]);
 		return;
-	default:
-		break;
 	}
 
-	/* The original process becomes the pager. */
+	/* The child process becomes the pager. */
 
 	close(fildes[1]);
 	if (dup2(fildes[0], STDIN_FILENO) == -1) {
@@ -970,6 +987,7 @@ spawn_pager(void)
 		    progname, strerror(errno));
 		exit((int)MANDOCLEVEL_SYSERR);
 	}
+	close(fildes[0]);
 
 	pager = getenv("MANPAGER");
 	if (pager == NULL || *pager == '\0')
--
 To unsubscribe send an email to source+unsubscribe@mdocml.bsd.lv