访问全局数组会导致分段错误

时间:2013-01-02 23:35:11

标签: c arrays pointers pthreads segmentation-fault

我正在尝试进行一项赋值,我将使用多个线程对文件中的输入进行排序,但是当我尝试使用结构数组来存储我需要在线程后恢复的信息时,我得到了一个分段故障。根据我的消息来源,我不确定为什么会导致这个错误。

这是主文件Threads.c.sess错误在for循环中,而起始行由注释设定。排序方法是我没有的另一个功能

#include "threads.h"
Threads* threadArray[4];
int linesPerThread;
int extraLines;

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

 if( argc != 4){
  printf("Wrong Number of Arguements!\n");
  return;
}
  n = atoi(argv[1]);
 char *inName = argv[2];



*threadArray = (Threads*) (*threadArray, n*sizeof(Threads));  




FILE* file = fopen(inName, "r");
 if(!file){
printf("invalid file Name \n");
return;}

int lines = 0;
char xyz[5000]; //makes fgets happy
while(fgets(xyz, 5000, file) != NULL){
  lines = lines+1;
}
fclose(file);
linesPerThread = lines / n;


 extraLines = lines - linesPerThread;

 int i =0;
 int methodCounter =1;


 printf("Right before Loop \n \n");

 for(i; i < n; i++){

   printf("first part of loop \n");
 \\The ling below here Seg Faults.
   (*threadArray + i)->id = i;

   printf("right after first ThreadArray access \n");
   if(methodCounter < 3){
 printf("method counter 1\n");
(*threadArray+i)->methodID = methodCounter;
 methodCounter++;
   }else{
 printf("method counter condition 2 \n");
(*threadArray + i)->methodID = 3;
   methodCounter = 1;}
   if(extraLines > 0){
 printf("extra Lines condition 1 \n");
(*threadArray+i)->lines = linesPerThread +1;
 extraLines= extraLines -1;
   }else{
 printf("extraLines condition 2 \n");
 (*threadArray+i)->lines = linesPerThread;
   }
   printf("Right before Thread Creation \n \n");
   pthread_t tID;
   pthread_create(&tID, NULL, sortMethod, (void*) &((*threadArray+i)->id));
   (*threadArray+i)->threadID = tID;
   printf("right after thread creation \n \n");
 }
 printf("right after loop \n \n");
 int c=0;

 printf("before thread joining \n");
 for(c; c< n; c++){
   pthread_join( (*threadArray+ c)->threadID, NULL);
 }


 }

这是头文件Threads.h

#include <sys/time.h>
#include <stdio.h> 
#include <pthread.h>
#include <stdlib.h>
#include <string.h>

typedef struct{
  int id;
  int lines;
  pthread_t threadID;
  int methodID;
}Threads;

void* sortMethod(void*ptr);
int main(int argc, char *argv[]);

非常感谢您提供的任何帮助。

1 个答案:

答案 0 :(得分:0)

在第

*threadArray = (Threads*) (*threadArray, n*sizeof(Threads));  

您将threadArray[0]设为(Threads*)(n*sizeof(Threads)。您可能希望该行中有realloccalloc

目前,

(*threadArray, n*sizeof(Threads))

是一个逗号表达式,该逗号表达式的值将强制转换为Threads*

由于您从未为threadArray

的任何元素分配内存
(*threadArray + i)->id = i;

取消引用无效指针。由于数组是静态的,因此指针最初被初始化为空指针,您只需将threadArray[0]设置为不同的值(但这也不会指向有效的内存)。