这样传递指针为什么不行？谢谢先

zphjita

运行断错误

#include <stdio.h> #include <pthread.h> #include <stdlib.h> struct arg_pthread {     FILE *fp;     int num;     pthread_mutex_t mutex; }; void *thread(void *arg) {     struct arg_pthread *arg_t = (struct arg_pthread *)arg;     int run;     FILE *fp = arg_t->fp;     if (pthread_mutex_lock(&(arg_t->mutex)) != 0)     {         perror("pthread_mutex_lock ");         return NULL;     }         for (run=1; run<4; run++) {         fprintf(fp, "In thread%d run %d\n", arg_t->num, run);     } unlock:     if (pthread_mutex_unlock(&(arg_t->mutex)) != 0)     {         perror("pthread_mutex_unlock ");         goto unlock;     } } int main() {     FILE *readf = NULL;     pthread_t myth1, myth2;     struct arg_pthread *arg;     readf = fopen("./testfile", "w+");     if (readf == NULL)     {         perror("fopen ");         exit(-1);     }     arg->fp = readf;     arg->num = 1;     if (pthread_mutex_init(&(arg->mutex), NULL) != 0)     {         perror("mutex init ");         fclose(readf);         exit(-1);     }     if (pthread_create(&myth1, NULL, thread, (void *)arg) != 0)     {         perror("pthread_create1 ");         fclose(readf);         exit(-1);     }         pthread_join(myth1, NULL);     arg->num = 2;     if (pthread_create(&myth2, NULL, thread, (void *)arg) != 0)     {         perror("pthread_create2 ");         fclose(readf);         pthread_cancel(myth1);         exit(-1);     }             pthread_join(myth2, NULL);     pthread_mutex_destroy(&(arg->mutex));     fclose(readf);     exit(0); }

hellioncu

struct arg_pthread arg;

arg.fp = readf;

if (pthread_create(&myth1, NULL, thread, (void *)&arg) != 0)

zphjita

我后头改为你这样的，但是我想知道为什么之前的不行，在这个问题上已经碰到两次了。

winddy

这就是传说中的野指针？

towardWang

原帖由 zphjita 于 2009-8-6 11:58 发表
我后头改为你这样的，但是我想知道为什么之前的不行，在这个问题上已经碰到两次了。

因为arg是指针，你没有给它分配空间就给它的成员fp赋值，所以段错误了。可以先用malloc之类的给arg分配空间。

zphjita

原来这样 谢谢

bsdc

正解。
我觉得小结构这样也不错啦
struct arg_pthread
{
    FILE *fp;
    int num;
    pthread_mutex_t mutex;
} arg_pthread;

struct arg_pthread *arg = &arg_pthread;


或者struct arg_pthread *arg = malloc(sizeof(struct arg_pthread));但要记得free。