は、なぜ、このコードは、セグメンテーション違反に

Question

を与えているこれはdubugging

プログラムはシグナルSIGSEGV、セグメンテーションフォールトを受信したエラーです。 [ スレッド0x7ffff6ff5700（LWP 17504）への切り替え] （q = 0x7ffff6ff4ec0、n = ...）discover.h：27 27
q-> TailCpuNode-> next = temp;プッシュの0x000000000040124cへの切り替え。

typedef struct CpuNode 
     { 
     struct CpuNode *next; 
     int cpuid; 
     }CpuNode; 

     typedef struct List 
     { struct CpuNode *HeadCpuNode; 
      struct CpuNode *TailCpuNode; 

     }List; 

     inline static void push(struct List *q,CpuNode *n){ 
     struct CpuNode *temp;      

     temp= (struct CpuNode *)malloc(sizeof(struct CpuNode)); 
     temp->cpuid=n->cpuid; 
     temp->next=NULL; 
     if (q->HeadCpuNode==NULL) q->HeadCpuNode=q->TailCpuNode=temp; 
     else { 
       q->TailCpuNode->next=temp; 
       q->TailCpuNode=temp; 
      } 
     } 

     void Discover(struct List *Acqcores){ 
     struct CpuNode *temp; 
     temp= (CpuNode *)malloc(sizeof(struct CpuNode)); 
     temp->cpuid=1; 
     push(Acqcores,temp); 
     } 



    int main(int argc, char **argv) { 
    struct List Acq_cores; 
    Discover(&Acq_cores); 
} 

Answer 1

あなたはAcq_coresを初期化していないので、あなたは初期化されていない変数にアクセスします。これは未定義の動作であり、クラッシュする可能性があります。

試してみてください。それ以外にも

struct List Acq_cores = {NULL, NULL}; 

あなたがメモリリークを持っています。これに変更します。一般的には

void Discover(struct List *Acqcores){ 
    struct CpuNode *temp; 
    temp= (CpuNode *)malloc(sizeof(struct CpuNode)); 
    temp->cpuid=1; 
    push(Acqcores,temp); 
    free(temp);   // Add this 
    } 

あなたがより良いだろう。

void Discover(struct List *Acqcores){ 
    struct CpuNode temp; 
    temp.cpuid=1; 
    push(Acqcores,&temp); 
    } 

参照のデモここhttp://ideone.com/rIYknh

Answer 2

次の行のコードに問題があります：

CpuNode temp; 
    temp= malloc(sizeof(CpuNode)); 

CpuNode *temp; 
    temp= (CpuNode *)malloc(sizeof(CpuNode)); 

温度の前にポインタを忘れた。

ここでは、segフォールトを排除したコードを示します。

inline static void push(List *q,CpuNode *n) 
    { 
    CpuNode *temp;      

    temp= (CpuNode *)malloc(sizeof(CpuNode)); 
    temp->cpuid=n->cpuid; 
    temp->next=NULL; 
    if (q->HeadCpuNode==NULL) 
     q->HeadCpuNode=q->TailCpuNode=temp; 
    else 
    { 
     q->TailCpuNode=temp; //This line should come first. 
     q->TailCpuNode->next=temp; 
    } 
} 

void Discover(List *Acqcores) 
{ 
    CpuNode *temp; 
    temp= malloc(sizeof(CpuNode)); 
    temp->cpuid=1; 
    push(Acqcores,temp); 
}