malloc上的双指针分段错误

Question

I'm making a program that reads in text from a file, delimits it into alphanumerical words, and so on. I'm running into an issue where, I've read from the file into a string, and I'm trying to delimit that string into a 2d array where each word is stored in an array. From there I will go on to order them.

However, I am getting a segfault when I malloc for words[n]. For a test file that I'm using, it has ~400 characters. No word is longer than say 10 characters, so (j - i + 1) wouldn't be a huge allocation that might cause a stack overflow (unless I'm misunderstanding my allocation).

I can explain my code for the function if need be, it's a manipulation of a strsplit() function that I wrote.

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

int main(int argc, char** argv) {
    if (argc != 2) {
        printf("Please provide 1 argument.\n");
        return 0;
    }

    FILE *file = fopen(argv[1], "r");
    fseek(file, 0, SEEK_END);
    size_t length = ftell(file); // Getting file size

    char buff, **words, buffer[length + 1];
    int i = 0, j = 0, n = 0, h = 0;

    buffer[length] = '\0';
    rewind(file);
    fread(buffer, 1, length, file); // Read file into buffer, close file
    fclose(file);

    // Deliminate to 2-D array of words
    while (buffer[i] != '\0') {
        while (isalnum(buffer[j]))
            j++;
        i = j;
        while (isalnum(buffer[j]))
            j++;

        words[n] = (char *)malloc(sizeof(char) * (j - i + 1));
        while (i < j)
            words[n][h++] = buffer[i++];

        n += 1, h = 0;
    }
}

It's worth noting that the file input works, and the algorithm to delimit works (it correctly captures the first word from the file from i to j). Do I need to malloc for **words? I don't think that's the issue, but I wouldn't know how many words are in a sample file so I would have to malloc some large amount.

Answer 1

You never actually set the value of words to anything. words needs some memory before you can populate words[n] .

You could do char** words = malloc(MAX_NUM_OF_WORDS * sizeof(char*)) or have a stack based collection: char* words[MAX_NUM_OF_WORDS]

Free tip for the day: using file without a NULL check is not good practice.