字符指针和 malloc

Question

我对 char 指针的概念有点困惑，所以我编写了一个简单的代码，只打印用户（我）提供的我的名字。我也想练习 malloc，所以我引用了指向 RAM 中某个内存的指针，但我真的不知道在 "sizeof(char) *" 后面放什么，因为那是用户输入，尚未决定。 另外，在这样做之后，我释放了内存，但我在命令行上收到一条错误消息：

*** Error in `./char': double free or corruption (fasttop): 0x00000000017fe030 ***
Aborted

好像我两次释放了相同的内存，但我不知道要删除或添加什么。请帮忙！

#include <stdio.h>
#include <cs50.h>

int main (void)
{

    char *strings = malloc(sizeof(char) * 10);


    printf("What is your name?\n");

    //wait for use to type his/her name
    strings = get_string();


    printf("Hello %s\n", strings);

    free (strings);

    return 0;
}

Answer 1

strings = get_string(); 行实际上将get_string() 返回的值分配给strings。它不会将其写入您分配的内存中。

所以malloc() 返回的值已被覆盖（在这种情况下丢失了）。

free(strings) 正在释放 get_string() 返回的任何内容。该问题没有提供代码，但可能对free() 无效。

因为运行时告诉你它被释放了两次，我猜你已经在 get_string() 分配了内存然后释放它并返回一个无效的指针。

如果你想使用你分配的内存你需要改变get_string()来接受一个指针：

void get_string(char *str){
    //Do whatever writing you value into str[] as an array of char..
}

好的做法应该是：

void get_string(char *str, size_t max){
    //Do whatever writing you value into str[] as an array of char..
    //Use max to avoid writing beyond the end of the space allocated...
}

然后调用get_string(strings,10);。

编辑：经过一番研究，该缺陷已被确定。 get_string() 不直接 free() 它返回的字符串，而是将其添加到由库进行的分配列表中，这些分配在退出时被释放（在一个名为 teardown() 的函数中注册了 atexit() 或其他编译器相关功能） .

这是一个糟糕的设计，因为没有提供消费者代码本身释放内存的安全方法，在典型的用例中，整个应用程序执行不需要这种方法。 get_double() 更糟，因为它从不返回分配的数据，但从不重复使用它，这相当于直接的内存泄漏。

代码应该：

1. 遵守文档并要求消费者代码为free() 字符串（为清楚起见，可能将其重命名为get_string_alloc()）。
2. 提供一个库例程来释放字符串（get_new_string() 和 release_string()）

在 C 中没有很好的方法来转移分配内存的所有权，但在剩余的执行过程中保留它绝对不是答案。 许多图书馆四处走动，将分配推送到消费者代码上，但是当无法知道所需空间的完整大小时，这很繁重。

我建议将_alloc() 放在任何返回对象的函数的末尾，消费者代码稍后必须free()。

因此，所提出问题的答案是删除malloc() 和free()，因为库处理这两者。但是请注意，如果您的程序多次调用该函数以及其他内部依赖它的函数（如 get_double()），您可能会因为库处于死区而耗尽内存。

Answer 2

问题是您的get_strings 覆盖了您最初的malloc。指针值是一个值。通过将其等同于其他东西，您替换了您的malloc 值。

Answer 3

内存分配在语句：
strings = get_string();

你不必malloc它（char *strings = malloc(sizeof(char) * 10); )

没有malloc 也可以正常工作

Answer 4

首先你创建了一个动态内存，它会被 *strings 指向。但是随后您使用 *strings 指针指向本地字符串（来自 get_string() 函数）。当您调用 free 时，程序正在尝试删除本地（堆栈）引用并抛出错误。

要解决这个错误，程序应该是

#include <stdio.h>
#include <cs50.h>

int main (void)
{

char *strings = malloc(sizeof(char) * 10);


printf("What is your name?\n");

//wait for use to type his/her name
strcpy(strings, get_string());       // Use strcpy instead of assigning


printf("Hello %s\n", strings);

free (strings);

return 0;
}

Answer 5

您没有包含get_string() 的代码，但是您使用错误的返回值覆盖了strings。您传递给free() 的地址必须来自malloc()，并且您似乎违反了这一点（除了丢失您的10 个字节的原始返回地址）。

假设 get_string() 返回静态存储（即您不需要释放它），您可以在不涉及 malloc() 的情况下执行此操作。

如果你真的想要，这样的事情可能会奏效：

printf("What is your name?\n");
const char *name = get_string();
const size_t nlen = strlen(name);
char * const name_copy = malloc(nlen + 1);
if(name_copy != NULL)
{
  memcpy(name_copy, name, nlen + 1);
  printf("Hello %s (from my own memory!)\n", name_copy);
  free(name_copy);
}

这是相当复杂的，但你明白了。

Answer 6

char *strings;

• 不需要新的 malloc，因为从 get_string() 函数返回的字符串已经在堆上，您只需要获取指向第一个字符的指针。 (get_string() function reference)

strings = get_string();

printf("Hello %s\n", 字符串);

• 打印字符串后，您应该释放为其分配的内存，如 get_string() 函数参考中所述

在堆上存储字符串（通过 malloc）；调用者必须释放内存 避免泄漏。

我认为其他一切都很好，试试这个代码：

#include <stdio.h>
#include <cs50.h>

    int main (void)
    {
        char *strings;

        printf("What is your name?\n");

        //wait for use to type his/her name
        strings = get_string();

        printf("Hello %s\n", strings);

        free (strings);

        return 0;
    }