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

Question

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

#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以上，则最终的printf开始打印奇怪的串联输出。（见nwords[x] = "1234567890123456"; // garbage行）为什么会这样？我用malloc给了它200个字符的内存。

Answer 1

使用此语句

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

将单个字符分配给words[x]。然后你做

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

其向CCD_ 4写入多于一个字符。

---

此外，使用

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");