wait()/waitpid() devuelve 1, no pid de hijo, ¿cómo solucionarlo?

Question

esto es una continuación de How to squeeze in additional parameters to a reaper function when a parent is signalled to kill a child (c)?

En mi segador(), trato de obtener el PID hijo de los padres está a punto de terminar (la palabra no brutal aquí). pero wait() no devuelve el pid del niño; en su lugar, devuelve 1. No puedo encontrar un documento para el valor de devolución de 1 en cualquier lugar. ¿Algún aviso?

void reaper(int sig) 
{ 
    int status, killedpid; 

    while(killedpid = (/*waitpid(-1, &status, WNOHANG)*/wait(&status)) >= 0) 
    { 
     printf("reaper %d killed %d\n", getpid(), killedpid); 
    } 
} 

Mis resultados:

reaper 5933 killed 1 //actual child pid is 5936 

gracias de antemano!

Answer 1

Este es el clásico asignación en error condicional - la expresión se evalúa de la siguiente manera (porque la comparación tiene mayor precedence luego asignación):

if (killedpid = (wait(&status) >= 0)) { ... 

El killedpid obtendrá un valor de TRUE, que es 1 en C. Para evitar este uso paréntesis y compilar con altos niveles de advertencia como -Wall -pedantic:

if ((killedpid = wait(...)) >= 0) { ...