c - SIGCHLD シグナルのハンドラー

Question

コードが機能せず、ループします。エラーは、SIGCHLD シグナルのハンドラーである gestore メソッドにあると思います。ハンドラーを使用して SIGCHLD シグナルをキャプチャするのはこれが初めてです。このプログラムは、数字が argv[1] 回出現するまで、0 から argv[1] までをさりげなく抽出し続けます。明確でない場合は、質問の最後に置いた私の古いプログラムをテストできます。エラーを見つけるのを手伝ってもらえますか?

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <sys/wait.h>
#include <errno.h>
#include <string.h>
#include <unistd.h>
#include <signal.h>

int a;
void gestore(int segnale);
int main(int argc, char * argv[]){
    int n = atoi(argv[1]), i, pid;
    int * vec;
    vec = malloc((n+1)*sizeof(*vec));
    memset (vec, 0, sizeof(*vec));
    char * newargv[] = {argv[0], argv[1] , NULL};
    for(i = 0; i < n; i++){
        pid = fork();
        if (pid == 0)
            execve("./throw-dice", newargv, NULL);
        signal(SIGCHLD, gestore);
        vec[WEXITSTATUS(a)]++;
    }
    while(vec[i] != n){
        for(i = 1; i < n+1 && vec[i] != n; i++){
            if(vec[i] != 0){
                pid = fork();
                if (pid == 0)
                    execve("./throw-dice", newargv, NULL);
                signal(SIGCHLD, gestore);
                vec[WEXITSTATUS(a)]++;
            }
        }
    }
    printf("The value %d is appeared %d times!\n", i, vec[i]);
    while (wait(&a) != -1);
    free(vec);
}

void gestore(int segnale){
    signal(segnale, SIG_IGN);
    waitpid(WAIT_ANY, &a, WNOHANG);
    signal(segnale, gestore);
}

私の目標は、古いプログラム (動作する) を変更して、子の終了ステータスを取得する方法を変更することでした。「待機」との同期から、SIGCHLD シグナルを処理する gestore メソッドとの非同期まで。これは私の古いプログラムです:

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <sys/wait.h>
#include <errno.h>
#include <string.h>
#include <unistd.h>

int main(int argc, char * argv[]){
    int n = atoi(argv[1]), a, i, pid;
    int * vec;
    vec = malloc((n+1)*sizeof(*vec));
    memset (vec, 0, sizeof(*vec));
    char * newargv[] = {argv[0], argv[1] , NULL};
    for(i = 0; i < n; i++){
        pid = fork();
        if (pid == 0)
            execve("./throw-dice", newargv, NULL);
        wait(&a);
        vec[WEXITSTATUS(a)]++;
    }
    while(vec[i] != n){
        for(i = 1; i < n+1 && vec[i] != n; i++){
            if(vec[i] != 0){
                pid = fork();
                if (pid == 0)
                    execve("./throw-dice", newargv, NULL);
                wait(&a);
                vec[WEXITSTATUS(a)]++;
            }
        }
    }
    printf("The value %d is appeared %d times\n", i, vec[i]);
    while (wait(&a) != -1);
    free(vec);
}

//throw-dice.c

#include <unistd.h>
#include <stdio.h>
#include <stdlib.h>

int main(int argc, char * argv[]) {
    int n, val;
    // Must have an argument
    if (argc < 2) {
        exit(-1);
    }
    // the 1st argument must be a positive number
    if ((n = atoi(argv[1])) <= 0) {
        exit(-1);
    }
    //  sleep(1); // sleep a bit
    srand(getpid()); // initialize the random seed with PID
    val = rand() % n + 1;
    printf("(PID=%d): got number %d\n", getpid(), val);
    exit(val);
}