免费();和马洛克();不断崩溃(C)

Question

我构建了这段代码来练习指针，程序一直 crashing.it 当我输入一个大数字来counter时它似乎崩溃了。 1-5 显然不会影响它，但是当你输入 30 时，它会不断崩溃，有时在分配本身malloc(...有时在 free(names[i]); 函数中。

这里有什么问题？

#include "stdafx.h"
#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <string.h>

int main() {
    char **names;
    char buffer[100];
    int i, bufferLen, counter;
    printf("how many names? ");
    scanf_s("%d", &counter);
    if (counter < 0) {
        printf("wrong choicen");
        return 1;
    }
    names = (char**)malloc(77 * sizeof(char));
    if (names == NULL) {
        printf("failed...n");
        return 1;
    }
    for (i = 0; i < counter; i++) { 
        printf("write the name!! (up to 100 chars): n");
        gets_s(buffer, sizeof(char) * 100);
        bufferLen = strlen(buffer) + 1;
        names[i] = (char*)malloc(sizeof(char)*bufferLen);
        if (names[i] == NULL) {
            printf("failed...n");
            return 1;
        }
        strcpy_s(names[i], sizeof(char)*bufferLen, buffer);
    }
    for (i = counter-1; i >= 0; i--) { //print names
        printf("no. %d, ptr no. %d (size: %d bytes): n", i+1, (int)(names[i]), sizeof(names[i]));
        puts(names[i]);
    }
    for (i = 0; i < counter; i++) { 
        if (names[i] != NULL)
            free(names[i]);
    }
    if (names != NULL)
        free(names);
    return 0;
}

Answer 1

这个：

names = (char**)malloc(77 * sizeof(char));

是错误的，sizeof (char)是 1，这不是你想要的。

它应该是：

names = malloc(77 * sizeof *names);

这与77 * sizeof (char *)相同，因为names char **这使得*names的类型char *。

演员阵容不是必需的，在我看来应该省略。

当然，使用文字77而不是数组长度count是非常奇怪的(和明显的代码气味(。

Answer 2

你可能想要names = (char**)malloc(counter * sizeof(char*)); .

此外，free处理空指针，无需在调用指针之前检查指针是否为 null。