c - не могу понять pthread_join()

Я не могу понять, где я не прав, после запуска прибыл код для того, где он запускает pthread_join(), много pthread_join() вернуть значение 3 вместо 0. Кроме того, печать значения i не всегда соответствует, и это приводит к ошибке сегментации и печати несколько раз с одной и той же позиции. Код, измененный в соответствии с требованиями в комментариях, все включает в себя для других частей программы. Тестирование только этого фрагмента кода создает ошибку сегментации при ошибке 3 на pthread_join()

#define _POSIX_C_SOURCE 200809L
#include <stdio.h>
#include <unistd.h>
#include <stdlib.h>
#include <assert.h>
#include <string.h>
#include <signal.h>
#include <pthread.h>

#include <errno.h>
#include <config.h>
#include <sys/select.h>
#include <ctype.h>
#include <sys/types.h> 
#include <sys/socket.h>
#include <sys/un.h>

void *threadF(){
    printf("hello\n");
    pthread_exit((void*)0);     
}




int main(int argc, char *argv[]) {

    FILE *fileconf=fopen(argv[2],"r"); 
    if(fileconf==NULL){
        fprintf(stderr, "Fopen\n",argv[2]);
        return -1;
    }
    set_conf(fileconf); //parse fileconf and set THREADSINPOOL correctly

    pthread_t array[THREADSINPOOL];
    int i,err,s=0;
    for(i=0;i<THREADSINPOOL;i++){
        if((err=pthread_create(&array[i],NULL,&threadF,NULL))!=0){
            fprintf(stderr,"thread\n");
            exit(errno);
        }
    }

    int tmp;

    for(i=0;i<THREADSINPOOL;i++){
        tmp=pthread_join(array[i],(void *)&s);
        printf("thread: %lu terminated\n tmp: %d\n",array[i],tmp);
    }

    return 0;
}

1 ответ

Решение

Проблема в том, что вы передаете адрес int к функции, которая ожидает адрес void *, В 64-битной системе есть хороший шанс, что int только 32-битный, тогда как void * является 64-битным. Так pthread_join заканчивает тем, что записывает 64-битные данные в место, которое достаточно велико для 32-битных систем. В результате вы перезаписываете память, которая не должна изменяться, и следует всякое неопределенное поведение.

Вот способ написать код, чтобы второй аргумент pthread_join на самом деле указатель на void *

for (i = 0; i < THREADSINPOOL; i++)
{
    void *value;
    if (pthread_join(array[i], &value) == 0)
        printf("thread %d returned %" PRIiPTR "\n", i, (intptr_t)value);
    else
        printf("thread %d failed\n", i);
}
Другие вопросы по тегам