c - Malloc 2D数组可能的大小溢出

这是我的程序的简化版本。

#include <stdio.h>

int main(void) {

    char **words, **nwords;

    int x;
    words = malloc(sizeof *words * 1000);
    if (words) {
        for (x = 0; x < 20; x++) {
            words[x] = malloc(sizeof *words[x] * 1);
        }
    }

    nwords = malloc(sizeof *nwords * 1000);
    if (nwords) {
        for (x = 0; x < 20; x++) {
            nwords[x] = malloc(sizeof(char) * 200);
            //nwords[x] = "123456789012345"; // works correctly
            nwords[x] = "1234567890123456"; // garbage
        }
    }

    int i;
    for (i=0; i<10;i++) {
        sprintf(words[i],"%s",nwords[i]);
    }

    for (i=0; i<10;i++) {
        printf("\n words[%d] = %s",i,words[i]);
    }

    return 0;
}

如果nwords[x]中的字符数增加到15个以上，那么finalprintf将开始打印奇怪的连接输出。（参见第nwords[x] = "1234567890123456"; // garbage行）为什么会这样？我用malloc给了它200个字符的内存。

最佳答案

用这句话

words[x] = malloc(sizeof *words[x] * 1);

为words[x]分配一个字符。那你就知道了

sprintf(words[i],"%s",nwords[i]);

它向words[i]写入多个字符。
还有，用

nwords[x] = malloc(sizeof(char) * 200);
nwords[x] = "1234567890123456"; // garbage

首先让nwords[x]指向堆外分配的一些内存，然后直接让它指向一个字符串文本。你也可能想要

nwords[x] = malloc(sizeof(char) * 200);
strcpy(nwords[x], "1234567890123456");

或

nwords[x] = strdup("1234567890123456");

关于c - Malloc 2D数组可能的大小溢出，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/20235412/