如何用char*修复此内存泄漏

How can I fix this memory leak with char*?

本文关键字:内存 泄漏 何用 char      更新时间:2023-10-16

我正在写一种简短的方法来获取达尔文上的MAC地址。但是,我不断收到以下错误消息:

malloc: can't allocate region
*** mach_vm_map(size=3907498536060022784) failed (error code=3)
malloc: *** set a breakpoint in malloc_error_break to debug

我认为这是由于内存泄漏所致。

我尝试通过引用传递char* addr变量,没有成功。

我的方法是:

char* getPhysicalAddress() {
    char* addr;
    #ifdef __APPLE__
        FILE *fp = popen("ifconfig en0 | awk '/ether/{print $2}'", "r");
        if (fp != NULL) {
            fscanf(fp, "%s", addr);
            pclose(fp);
        }
        else {
            addr = "[unknown]";
        }
    #endif
    #ifdef __linux__
        FILE* file = popen("cat /sys/class/net/eth0/address", "r");
        if (file != NULL) {
            fscanf(file, "%s", addr);
            pclose(file);
        }
        else {
            addr = "[unknown]";
        }
    #endif
    return addr;
}

,在我的主要功能中,我用char* addr = getPhysicalAddress();调用该功能。

如何修复此方法中的内存泄漏?

让我们看一下此代码:

char* addr;
...
fscanf(fp, "%s", addr);

看起来,从此代码中,您想从fp文件中读取到字符串addr。但是,如书面,此代码导致不确定的行为(哦,不!(,并且在您的情况下,它正在损坏您的内存管理器,因此malloc错误。

那是怎么回事?请记住,addr并不是真正的字符串。相反,这是一个char*,这意味着它是一个变量,上面写着:"请查看我要指向的位置,您会找到存储一些字符的空间。"但是,您已经将其保留了非直接化,这意味着它指向内存中的某个随机区域,这可能是您实际上没有的空间。当fscanf试图写入该位置的字符时,它会偶然地在该位置发生任何记忆,这是一件坏事。

换句话说,这是不是内存泄漏。相反,这是一个非初始化的指针写。

有几个方法可以解决此问题。一种选择是将addrchar*转换为实际字符数组,这意味着addr本身将表示"这里有一些放置某些字符的空间"。您需要调整fscanf调用,以确保您没有阅读太多字符,否则您可能会溢出缓冲区。

另外,由于您已经使用C 标记了此标签,因此另一个选项是使用std::string之类的类型来保存内存,然后使用循环从fp中拉出字符,将其附加到该std::string对象。

希望这会有所帮助!