Skip to content

Commit

Permalink
* sysdeps/posix/system.c: Remove NO_WAITPID support, it cannot
Browse files Browse the repository at this point in the history
	really work anyway.
  • Loading branch information
Ulrich Drepper committed Jan 17, 2007
1 parent 000b027 commit d78bce1
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 17 deletions.
5 changes: 5 additions & 0 deletions ChangeLog
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
2007-01-16 Ulrich Drepper <drepper@redhat.com>

* sysdeps/posix/system.c: Remove NO_WAITPID support, it cannot
really work anyway.

2007-01-15 Ulrich Drepper <drepper@redhat.com>

* nscd/connections.c (servinfo): Renamed to reqinfo. Change all
Expand Down
18 changes: 1 addition & 17 deletions sysdeps/posix/system.c
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
/* Copyright (C) 1991-2000, 2002, 2003, 2005 Free Software Foundation, Inc.
/* Copyright (C) 1991-2000,2002,2003,2005,2007 Free Software Foundation, Inc.
This file is part of the GNU C Library.
The GNU C Library is free software; you can redistribute it and/or
Expand Down Expand Up @@ -143,27 +143,11 @@ do_system (const char *line)
else
/* Parent side. */
{
#ifdef NO_WAITPID
pid_t child;
do
{
child = __wait (&status);
if (child <= -1 && errno != EINTR)
{
status = -1;
break;
}
/* Note that pid cannot be <= -1 and therefore the loop continues
when __wait returned with EINTR. */
}
while (child != pid);
#else
/* Note the system() is a cancellation point. But since we call
waitpid() which itself is a cancellation point we do not
have to do anything here. */
if (TEMP_FAILURE_RETRY (__waitpid (pid, &status, 0)) != pid)
status = -1;
#endif
}

#ifdef CLEANUP_HANDLER
Expand Down

0 comments on commit d78bce1

Please sign in to comment.