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);
}