Code

make git-shell paranoid about closed stdin/stdout/stderr
authorPaolo Bonzini <bonzini@gnu.org>
Wed, 27 Aug 2008 15:20:35 +0000 (17:20 +0200)
committerJunio C Hamano <gitster@pobox.com>
Fri, 29 Aug 2008 07:14:29 +0000 (00:14 -0700)
commit0cfeed2e1d320cc76c434e0bfc26d90065754e46
tree6d97516c7eff97cc960288e0930552d75ad54394
parent29f28151c5ac0ed842e9a00438f1930ee4052757
make git-shell paranoid about closed stdin/stdout/stderr

It is in general unsafe to start a program with one or more of file
descriptors 0/1/2 closed.  Karl Chen for example noticed that stat_command
does this in order to rename a pipe file descriptor to 0:

    dup2(from, 0);
    close(from);

... but if stdin was closed (for example) from == 0, so that

    dup2(0, 0);
    close(0);

just ends up closing the pipe.  Another extremely rare but nasty problem
would occur if an "important" file ends up in file descriptor 2, and is
corrupted by a call to die().

Fixing this in git was considered to be overkill, so this patch works
around it only for git-shell.  The fix is simply to open all the "low"
descriptors to /dev/null in main.

Signed-off-by: Paolo Bonzini <bonzini@gnu.org>
Acked-by: Stephen R. van den Berg <srb@cuci.nl>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
shell.c