函数重新分配c字符串后delete []时出现无效指针错误

Question

我正在尝试使用“new char[]”删除[] 之前由另一个函数分配的 c 字符串 (char*)。但是我得到一个无效的指针错误。

首先是代码

类：

class myclass
{
    public:
        myclass(const char*);
        ~myclass();
        char* getNum(const char*, myclass*);
    private:
        sqlite3* db; //db object
        int rc; //int returned by sqlite3 functions
        char* zErrMsg; //error message
        char* sql; //sql statement
        char* num;                 //<- this is it
        static int callback(void*, int, char**, char**);
};

一些功能：

myclass::myclass(const char* filename)
{
    //allocating  
    num = new char[2];
    num[0] = 'i';
    num[1] = '\0';
    //connect to sqlite db using filename
}

char* myclass::getNum(const char* id, myclass* obj) //obj will be passed to callback function
{
    std::cout<<"getNum"<<std::endl;
    std::cout<<static_cast<void *>(&num[0])<<"("<<num[0]<<")"<<std::endl; //test1. writes address and first char. all seems ok
    delete[] num; //here is the problem
    num = new char[10];
    strcpy(num, "not found");
    std::cout<<static_cast<void *>(&num[0])<<"("<<num[0]<<")"<<std::endl; //test2. writes same address as test1 and first char. all seems ok
    //sql query construction. Nothing wrong here.
    //next query gets executed and callback function gets called.
    rc = sqlite3_exec(db, query, callback, static_cast<void*>(obj), &zErrMsg);
    return num; //returns num from db. "not found" otherwise
}

int myclass::callback(void* obj, int argc, char** argv, char** azColName)
{
    std::cout<<"callback"<<std::endl;
    myclass* const object = static_cast<myclass*>(obj);
    std::cout<<static_cast<void *>(&object->num[0])<<"("<<object->num[0]<<")"<<std::endl; //test3. writes same address as test2 and first char. all seems ok
    delete[] object->num; //no problems here
    object->num = new char[strlen(argv[0])+1]; //address of num changes
    object->num = argv[0];
    std::cout<<static_cast<void *>(&object->num[0])<<"("<<object->num[0]<<")"<<std::endl; //test4. writes different address and first char. But all seems ok.
    return 0;
}

关于回调：似乎 sqlite3_exec() 想要一个静态回调函数。所以它看不到“myclass”的“num”成员，因为它不是静态的。而且我不能使它成为静态的，因为我需要它用于其他成员函数。所以我将对象地址传递给 in 并使用 -> 来访问它的成员。

主要：

int main(int argc, char const *argv[])
{
    myclass a(filename);
    a.getNum("id", &a) //works first time
    a.getNum("id", &a) //doesn't work
    return 0;
}

第一次调用 getNum() 可以完成工作。没有什么不对。 第二个（第一次回调修改num后）我得到一个无效的指针错误。

它发生在 getNum 函数内的第一个 delete[] 上。如果我尝试在它之前输出 num - 一切正常并且我得到正确的输出。

我知道这与回调函数内部的重新分配有关，但我不明白是什么。我必须动态重新分配它，因为我不知道从 db 返回的值的长度。 我在 getNum 中取消分配它，因为我不想要任何内存泄漏。 如果我删除该释放（在 getNum 内），程序似乎可以完美运行（但可能存在内存泄漏？）。

我

Answer 1

在您的回调中，您将 num 从不是通过 new 分配的 sqlite 重新分配给 argv[0]。

object->num = new char[strlen(argv[0])+1]; //address of num changes
object->num = argv[0]; // <--- here

你的意思可能是：

object->num = new char[strlen(argv[0])+1]; //address of num changes
strcpy (object->num, argv[0]);

Answer 2

object->num = argv[0]; 你在这里重新分配你的指针。因此，您尝试删除 argv[0]。