Ошибка сегмента для программы массива C

Question

По какой-то причине я получаю ошибку сегмента в строке let [i] = all (4,4, 'a', 'z'); Я не могу понять, почему. Я пытаюсь поместить aaaa-zzzz в массив &, а затем сравнить значения один за другим с зашифрованным паролем. Любые советы помогут. Благодарю. Ive дал два фрагмента кода, которые дают эту проблему.

char* all(int a, int n,char begin, char end){ 
    int i, j; 
    int k = 0; 
    char *c = malloc((n+1)*sizeof(char)); 

    char msg[] = ""; 
    k = strlen(msg); 
    for(i = a; i <= n; i++) 
    { 
     for(j=0;j<i;j++) c[j]=begin; 
     c[i]=0; 
     do { 
     for(int g = 0; g < k; g++) 
      msg[g] = *c; 
     } while(inc(c,begin,end)); 
    } 
    return c; 
    free(c); 
} 

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

    int f = open("pass.txt", O_RDONLY); 
    if (f < 0) 
     return 0; 
    char buf[1024]; 
    while (my_fgets(buf, sizeof(buf), f)) 
    { 
     int i = 0; 
      let[i] = all(4,4,'a','z'); 

     result = crypt(argv[i+1], buf); 
     i++; 
     int ok = strcmp (result, buf) == 0; 

     return ok ? 0 : 1; 

    } 
    free(let); 
    close(f); 
} 

Полная программа может быть найдена здесь:

http://pastie.org/10035996

Answer 1

Вы хотите let быть массивом (указатели на отдельные) строк. Вы выделяете пространство для хранения строк, но не для самого массива. Так что, либо объявить:

char *let[MAX_STRINGS]; 

, если у Вас есть максимальное количество строк, известных во время компиляции, или передать его динамически:

let = malloc (sizeof (char *) * whatever_number); 

В обоих случаях вы должны заботиться, чтобы не превышать выделить количество во время фазы ввода. Если вы хотите иметь возможность обрабатывать любое количество входных строк, вам придется изменить структуру данных, возможно, в связанном списке, но я предполагаю, что это передовая тема для вас на данный момент.

Также обратите внимание, что free(let), который у вас есть прямо сейчас, в вашем коде будет освобождать память только для самого массива, но вам также необходимо освободить отдельные строки до этого. В основном вы должны выполнить free() за каждые malloc(), которые у вас есть в вашем коде.