diff options
author | Sami Kerola | 2014-04-26 19:30:06 +0200 |
---|---|---|
committer | Karel Zak | 2014-05-06 10:18:37 +0200 |
commit | dc64469f4f821831b4c7c4413f85829a3eb32f13 (patch) | |
tree | 40c356ec4a0e2dc5eecedd2f8821b255eb03d191 /lib/pager.c | |
parent | lib/timeutils: fix memory leak (diff) | |
download | kernel-qcow2-util-linux-dc64469f4f821831b4c7c4413f85829a3eb32f13.tar.gz kernel-qcow2-util-linux-dc64469f4f821831b4c7c4413f85829a3eb32f13.tar.xz kernel-qcow2-util-linux-dc64469f4f821831b4c7c4413f85829a3eb32f13.zip |
lib/pager: use names when referring to standard file descriptors
Signed-off-by: Sami Kerola <kerolasa@iki.fi>
Diffstat (limited to 'lib/pager.c')
-rw-r--r-- | lib/pager.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/lib/pager.c b/lib/pager.c index c6e74e899..9e09cd52b 100644 --- a/lib/pager.c +++ b/lib/pager.c @@ -63,10 +63,10 @@ static int start_command(struct child_process *cmd) cmd->pid = fork(); if (!cmd->pid) { if (need_in) { - dup2(fdin[0], 0); + dup2(fdin[0], STDIN_FILENO); close_pair(fdin); } else if (cmd->in > 0) { - dup2(cmd->in, 0); + dup2(cmd->in, STDIN_FILENO); close(cmd->in); } @@ -134,7 +134,7 @@ static void pager_preexec(void) fd_set in; FD_ZERO(&in); - FD_SET(0, &in); + FD_SET(STDIN_FILENO, &in); select(1, &in, NULL, &in, NULL); setenv("LESS", "FRSX", 0); @@ -145,8 +145,8 @@ static void wait_for_pager(void) fflush(stdout); fflush(stderr); /* signal EOF to pager */ - close(1); - close(2); + close(STDOUT_FILENO); + close(STDERR_FILENO); finish_command(&pager_process); } @@ -160,7 +160,7 @@ void setup_pager(void) { const char *pager = getenv("PAGER"); - if (!isatty(1)) + if (!isatty(STDOUT_FILENO)) return; if (!pager) @@ -178,9 +178,9 @@ void setup_pager(void) return; /* original process continues, but writes to the pipe */ - dup2(pager_process.in, 1); - if (isatty(2)) - dup2(pager_process.in, 2); + dup2(pager_process.in, STDOUT_FILENO); + if (isatty(STDERR_FILENO)) + dup2(pager_process.in, STDERR_FILENO); close(pager_process.in); /* this makes sure that the parent terminates after the pager */ |