5

我有记忆问题。我以这种方式使用结构：

包.h 文件

#pragma once
#include <cstdlib>

struct Package {
    char *data;
    long long int *packageNumber;
    long long int *allPackages;

    Package(const int sizeOfData);
    ~Package();
};

包.cpp

#include "Package.h"

Package::Package(const int sizeOfData) {
    void *ptr = malloc(2 * sizeof(long long int) + sizeOfData * sizeof(char));
    packageNumber = (long long int*) ptr;
    allPackages = (long long int*) ((long long int*)ptr + sizeof(long long int));
    data = (char*)((char*)ptr + 2 * sizeof(long long int));
}

Package::~Package() {
    free(data);
    free(packageNumber);
    free(allPackages);
}

在方法中：

for (int j = 0; j < this->bufforSize || i * bufforSize + j < allPackages; j++) {
            Package package(this->packageSize);
            this->file->read(package.data, this->packageSize);
            *package.allPackages = allPackages;
            *package.packageNumber = i * this->bufforSize + j;
            this->dataPacked->push_back(package);
        }

在括号结束后它会抛出错误："HEAP[zad2.exe]: Invalid address specified to RtlValidateHeap( 00000056FEFE0000, 00000056FEFF3B20 )" 我不知道我做错了什么。请帮忙，迈克尔。

编辑：现在它正在为循环的第一次迭代工作。帮助我将析构函数更改为：

Package::~Package() {
    free(packageNumber);
}

但是现在析构函数在循环的第二次迭代中对同一个结构对象执行了两次。

4

2 回答 2

7

阅读描述free：_

如果 ptr 的值不等于先前由 std::malloc()、std::calloc() 或 std::realloc() 返回的值，则行为未定义。

然后看看你的代码，注意我添加的注释。

void *ptr = malloc(2 * sizeof(long long int) + sizeOfData * sizeof(char));
packageNumber = (long long int*) ptr; // you got this from malloc
allPackages = (long long int*) ((long long int*)ptr + sizeof(long long int)); // the value of this pointer is not equal to anything returned by malloc
data = (char*)((char*)ptr + 2 * sizeof(long long int)); // the value of this pointer is not equal to anything returned by malloc either

最后在析构函数中：

free(data); // was not allocated with malloc -> undefined behaviour
free(packageNumber); // was allocated with malloc -> OK
free(allPackages); // was not allocated with malloc -> undefined behaviour

您尝试删除未从malloc. 这会导致未定义的行为。该错误是由于未定义的行为。请意识到这会free(packageNumber)释放分配给malloc. data这包括和指向的内存allPackages。

有一个简单的经验法则：free每次调用malloc/时只调用一次calloc。同样适用于delete+new和delete[]+ new[]。

于 2015-11-09T16:30:18.733 回答
0 
allPackages = (long long int*) ((long long int*)ptr + sizeof(long long int));

当你使用 long long int 指针（在我们的例子中它是被强制转换后的 ptr）并且你想要提前 sizeof(long long int) 字节时，你只需要做 ptr++;

但我建议您重写代码并使用 3 个 malloc 而不是一个。

于 2015-11-09T16:15:50.277 回答