Browse Source

check for error return from fork()

Trivial fixup for fork() callsites which do not check for errors.

Signed-off-by: Paul T Darga <pdarga@umich.edu>
Signed-off-by: Junio C Hamano <junkio@cox.net>
maint
Paul T Darga 19 years ago committed by Junio C Hamano
parent
commit
c9bc159d7f
  1. 2
      connect.c
  2. 6
      imap-send.c
  3. 6
      rsh.c

2
connect.c

@ -657,6 +657,8 @@ int git_connect(int fd[2], char *url, const char *prog)
if (pipe(pipefd[0]) < 0 || pipe(pipefd[1]) < 0) if (pipe(pipefd[0]) < 0 || pipe(pipefd[1]) < 0)
die("unable to create pipe pair for communication"); die("unable to create pipe pair for communication");
pid = fork(); pid = fork();
if (pid < 0)
die("unable to fork");
if (!pid) { if (!pid) {
snprintf(command, sizeof(command), "%s %s", prog, snprintf(command, sizeof(command), "%s %s", prog,
sq_quote(path)); sq_quote(path));

6
imap-send.c

@ -924,6 +924,7 @@ imap_open_store( imap_server_conf_t *srvc )
struct hostent *he; struct hostent *he;
struct sockaddr_in addr; struct sockaddr_in addr;
int s, a[2], preauth; int s, a[2], preauth;
pid_t pid;


ctx = xcalloc( sizeof(*ctx), 1 ); ctx = xcalloc( sizeof(*ctx), 1 );


@ -941,7 +942,10 @@ imap_open_store( imap_server_conf_t *srvc )
exit( 1 ); exit( 1 );
} }


if (fork() == 0) { pid = fork();
if (pid < 0)
_exit( 127 );
if (!pid) {
if (dup2( a[0], 0 ) == -1 || dup2( a[0], 1 ) == -1) if (dup2( a[0], 0 ) == -1 || dup2( a[0], 1 ) == -1)
_exit( 127 ); _exit( 127 );
close( a[0] ); close( a[0] );

6
rsh.c

@ -48,6 +48,7 @@ int setup_connection(int *fd_in, int *fd_out, const char *remote_prog,
int sizen; int sizen;
int of; int of;
int i; int i;
pid_t pid;


if (!strcmp(url, "-")) { if (!strcmp(url, "-")) {
*fd_in = 0; *fd_in = 0;
@ -91,7 +92,10 @@ int setup_connection(int *fd_in, int *fd_out, const char *remote_prog,
if (socketpair(AF_UNIX, SOCK_STREAM, 0, sv)) if (socketpair(AF_UNIX, SOCK_STREAM, 0, sv))
return error("Couldn't create socket"); return error("Couldn't create socket");


if (!fork()) { pid = fork();
if (pid < 0)
return error("Couldn't fork");
if (!pid) {
const char *ssh, *ssh_basename; const char *ssh, *ssh_basename;
ssh = getenv("GIT_SSH"); ssh = getenv("GIT_SSH");
if (!ssh) ssh = "ssh"; if (!ssh) ssh = "ssh";

Loading…
Cancel
Save