Bu C programı neden bu konumda segfaultedir?

Question

Temelde bir cümleyi alıp her kelime içinde kaç kere göründüğü sayabilir bir C programı yapmaya çalışıyorum. Sorunu tam olarak yeniden üreten soyulmuş bir sürüm oluşturdum.

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 

typedef struct testStrut{ 
    char * string; 
    int uses; 
}word; 

void grow(); 

int main(){ 
    int i; 
    int count = 1; 
    word ** words; 

    words = (word **) malloc(count * sizeof(word *)); 
    words[0] = (word *) malloc(sizeof(word)); 

    for(i = 0; i < 10; i++){ 
     printf("Re-looping i: %d \n", i); 
     printf("words[0]->string = %s \n", words[0]->string); 
     grow("TEST", words, &count); 
    } 

    printf("Done."); 
    return 0; 
} 

void grow(char * str, word ** words, int * count){ 
    word ** tmp; 

    tmp = realloc(words, (*count) * sizeof(word *)); 

    if(tmp){ 
     tmp[(*count)-1] = malloc(sizeof(word)); 
     tmp[(*count)-1]->string = malloc(strlen(str)+1); /*+1 for null terminator as pointed out */ 
     strcpy(tmp[(*count)-1]->string, str); 
     tmp[(*count)-1]->uses = 1;  
     words = tmp; 
     (*count)++; 
    } else{ 
     printf("Failure to allocate. \n"); 
     exit(0); 
    } 
    printf("Count: %d and word[0] %s \n", (*count), str); 
} 

yanı sıra bir çalışma çıktısı:

Re-looping i: 0 
words[0]->string = (null) 
Count: 2 and word[0] TEST 
Re-looping i: 1 
words[0]->string = TEST 
Count: 3 and word[0] TEST 
Re-looping i: 2 
words[0]->string = TEST 
Count: 4 and word[0] TEST 
Re-looping i: 3 
words[0]->string = TEST 
Count: 5 and word[0] TEST   /*Prints it fine? */ 
Re-looping i: 4 
Segmentation fault (core dumped)  /*Suddenly unable to print it? */ 

Yetiştirme fonksiyonunu sona erdiren ve arasındaki döngü içinde kelime [0] değerini yeniden gidiş neden anlayamıyorum -> str aniden kayboldu. Kaybettiğim bir şey mi var?

[Ben de benim yöntemi prototip doğru biri değil fark malloc.I şey azat gerektiğini biliyoruz ama sadece benim sorunu göstermiştir hızlı bir program yapmak istediğini] İlk olarak

Answer 1

sorun burada ... biraz daha basit hale getirdik:

words = tmp; 

Bu ifade fonksiyonu dışında hiçbir etkisi yoktur.

(veya olmayabilir) Başka bir hafıza konumuna yeni bir işaretçi döndürebilir realloc olduğundan, kod çarpmadan önce bir kaç kez çalışabilir.

yerine kelimenin ** parametre arasında *** bir kelime kullanmak ya da sadece dönen yeni işaretçi olmalıdır:

word** grow(char * str, word ** words, int * count){ 
    word ** tmp; 

    tmp = realloc(words, (*count) * sizeof(word *)); 

    if(tmp){ 
     tmp[(*count)-1] = malloc(sizeof(word)); 
     tmp[(*count)-1]->string = malloc(strlen(str)+1); /*+1 for null terminator as pointed out */ 
     strcpy(tmp[(*count)-1]->string, str); 
     tmp[(*count)-1]->uses = 1; 
     (*count)++; 
    } else{ 
     printf("Failure to allocate. \n"); 
     exit(0); 
    } 
    printf("Count: %d and word[0] %s \n", (*count), str); 
    return tmp; 
} 

Ve bu şekilde diyoruz:

words = grow("TEST", words, &count); 

Answer 2

for döngüsünün yinelenmesi aşağıdaki satır başlatılmamış belleğe erişir.

printf("words[0]->string = %s \n", words[0]->string); 

Ayrıca

void grow(); 

bildirdikten ancak

void grow(char * str, word ** words, int * count) 

yani fiili imza Önce ihtiyaç bu satır yani önce grow aramak için. Ayrıca realloc ve varsayarak orijinal pointer ana noktalarda pointer words söyledi.

bu deneyin. Ben

#include <assert.h> 
#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
typedef struct testStrut{ 
    char * string; 
    int uses; 
} word; 

void grow(const char *str, word *words, int count); 

int main(){ 
    int i; 
    word * words; 
    printf("sizeof word == %zu\n", sizeof(word)); 
    assert(sizeof(word) == 16); 
    words = malloc(sizeof(word)); 
    for(i = 0; i < 10; i++){ 
    printf("Re-looping i: %d \n", i); 
    grow("TEST", words, i); 
    printf("words[0]->string = %s \n", words[0].string); 
    } 
    printf("Done."); 
    return 0; 
} 
void grow(const char * str, word *words, int count){ 
    word ** tmp; 
    int idx = count - 1; 
    printf("size == %zu\n", count * sizeof(word)); 
    tmp = realloc(words, count * sizeof(word)); 
    size_t str_len = strlen(str); 
    if(tmp != NULL) { 
    tmp[idx]   = malloc(sizeof(word*)); 
    tmp[idx]->string = malloc(str_len + 1); 
    strcpy(tmp[idx]->string, str); 
    tmp[idx]->string[4] = '\0'; 
    tmp[idx]->uses = 1; 
    } else{ 
    printf("Failure to allocate. \n"); 
    exit(0); 
    } 
    printf("Count: %d and word[0] %s \n", count, str); 
}