Question

我的代码就像这样

starts是DWORD32

的数组

threads是HANDLE

的数组

void initThreads(HANDLE* threads, int size)
{
    DWORD32* starts = (DWORD32*)malloc(sizeof(DWORD32) * size);
    for (int i = 0; i < size; ++i)
    {
        starts[i] = num_steps / numThreads * i;
    }
    for (int i = 0; i < size; ++i)  
    {
        DWORD32* para = starts + i;
        printf("create %ld\n", *para);
        threads[i] = CreateThread(NULL, 0, portionCal, (void*)para, 0, NULL);
    }
    free(starts);
}

DWORD WINAPI portionCal(LPVOID pArg) 
{ 
   double x, portionSum = 0.0;
   DWORD32 start = *(DWORD32*)pArg;
   printf("start at %d\n", start);
}

但结果是

create 0
create 25000000
start at 0
create 50000000
create 75000000
start at 50000000
start at -17891602
start at 25000000

为什么结果看起来像这样？

Answer 1

我们无法看到启动的范围，但这可以从失败中猜到。它可能是一个局部变量，在线程开始运行时早已消失。所以你只会阅读垃圾。你需要一个稳定的指针，从全局变量或malloc（）中获取一个。

编辑后：不要像这样调用free（）。它必须保持稳定，直到所有线程完成使用它。您可以考虑使用InterlockedDecrement（）来引用计数。

Answer 2

创建线程后立即释放starts数组。所以会发生的是线程传递指向内存的指针，这些内存可能在线程有机会读取之前被释放。如果发生这种情况，则结果行为未定义。

您可以通过确保指针引用的内存的生命周期超出线程的生命周期来解决问题。通常，您可以通过为堆分配每个线程的数据，并让线程在获取信息副本时调用free。

在这种情况下，解决问题的更简单方法是传递整数值而不是指向它的指针。像这样：

threads[i] = CreateThread(NULL, 0, portionCal, (void*)starts[i], 0, NULL);

在你的主题中：

DWORD32 start = (DWORD32)pArg;

通过CreateThread传递参数失败？

2 个答案: