删除指针有时会导致堆损坏

我有一个使用自定义线程池类运行的多线程应用程序。这些线程都执行相同的函数，但参数不同。

这些参数通过以下方式传递给线程池类：

// jobParams is a struct of int, double, etc...
jobParams* params = new jobParams;
params.value1 = 2;
params.value2 = 3;

int jobId = 0;

threadPool.addJob(jobId, params);

一旦线程无事可做，它就会获取下一个参数并运行作业函数。我决定删除线程池类中的参数：

ThreadPool::~ThreadPool() {
    for (int i = 0; i < this->jobs.size(); ++i) {
        delete this->jobs[i].params;
    }
}

但是，这样做时，我有时会遇到堆损坏错误：

指定给 RtlFreeHeap 的地址无效

奇怪的是，在一种情况下它工作得很好，但在另一种程序中它会因这个错误而崩溃。我尝试在其他位置删除指针：在执行作业函数之后的线程中（我得到相同的堆损坏错误）或在作业函数本身的末尾（在这种情况下没有错误）。

我不明白从不同的地方删除相同的指针（我检查过，地址是相同的）如何改变任何东西。这与它是多线程的事实有什么关系吗？

我确实有一个关键部分来处理对参数的访问。我认为问题不在于同步访问。无论如何，只有在所有线程完成后才会调用析构函数，并且我不会在其他任何地方删除任何指针。指针可以自动删除吗？

至于我的代码。作业列表是一个结构体的队列，由作业的id（用于稍后能够获取特定作业的输出）和参数组成。

getNextJob()每次线程完成执行上一个作业时都会调用它（它们有一个指向线程池的指针）。

void ThreadPool::addJob(int jobId, void* params) {
    jobData job; // jobData is a simple struct { int, void* }
    job.ID = jobId;
    job.params = params;

    // insert parameters in the list
    this->jobs.push(job);
}

jobData* ThreadPool::getNextJob() {    
    // get the data of the next job
    jobData* job = NULL;

    // we don't want to start a same job twice,
    // so we make sure that we are only one at a time in this part
    WaitForSingleObject(this->mutex, INFINITE);

    if (!this->jobs.empty())
    {
        job = &(this->jobs.front());
        this->jobs.pop();
    }

    // we're done with the exclusive part !
    ReleaseMutex(this->mutex);

    return job;
}

让我们把这个问题反过来看：你为什么要使用指针？

class Params
{
int value1, value2; // etc...
}

class ThreadJob
{
  int jobID;  // or whatever...
  Params params;
}

class ThreadPool
{
  std::list<ThreadJob> jobs;

  void addJob(int job, const Params & p)
  {
     ThreadJob j(job, p);
     jobs.push_back(j);
  }
}

没有新的、删除的或指针...显然，一些实现细节可能是错误的，但你可以了解整体情况。