Question

我正在用C语言编写一个Windows程序作为家庭作业，我遇到的问题导致我的程序崩溃，program.exe已停止工作。我认为这是由于内存未正确分配。

该程序应该启动多个线程来执行任务，我在MSDN上找到了创建线程的示例。我已将部分代码添加到我的程序中。

我的节目：

#include <stdio.h>
#include <stdlib.h>
#include <malloc.h>
#include <Windows.h>

#define MAX_THREADS 4
#define BUFFER_SIZE 65000

DWORD WINAPI SomeFunction( LPVOID lpParam );

char fileBuffer[BUFFER_SIZE];

typedef struct MyData {
    int val1;
    int val2;
} MYDATA, *PMYDATA;

int main(int argc, char *argv[])
{
    int i = 0;
    int j = 0;
    PMYDATA pDataArray[MAX_THREADS];
    DWORD   dwThreadIdArray[MAX_THREADS];
    HANDLE  hThreadArray[MAX_THREADS]; 


        for (i; i < MAX_THREADS; i++)
        {
            pDataArray[i] = (PMYDATA) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY,
                    sizeof(MYDATA));

            if( pDataArray[i] == NULL )
            {
               // If the array allocation fails, the system is out of memory
               // so there is no point in trying to print an error message.
               // Just terminate execution.
                ExitProcess(2);
            }

            // Create the thread to begin execution on its own.
            hThreadArray[i] = CreateThread(NULL, 0, SomeFunction, pDataArray[i], 0, &dwThreadIdArray[i]);

            if (hThreadArray[i] == NULL)
            {
                printf("%s\n", "Error creating thread!");
                ExitProcess(3);
            }

        }

        for (j; j < MAX_THREADS; j++)
        {
            printf("%s%d\n", "j=", j);
            WaitForSingleObject(hThreadArray[j], INFINITE);
        }

        //WaitForMultipleObjects(MAX_THREADS, hThreadArray, TRUE, INFINITE);
        i = 0;
        for(i; i<MAX_THREADS; i++)
        {
            CloseHandle(hThreadArray[i]);
            if(pDataArray[i] != NULL)
            {
                HeapFree(GetProcessHeap(), 0, pDataArray[i]);
                pDataArray[i] = NULL;    // Ensure address is not reused.
            }
        }

        printf("%s\n", "DONE!");

        return 0;

}

DWORD WINAPI SomeFunction( LPVOID lpParam)
{
    PMYDATA pDataArray;
    int anotherInt;

    anotherInt = pDataArray->val1; // PROBLEM OCCURS HERE!

    printf("%s%d\n", "Printing int ", anotherInt);


    return 0;
}

上面的程序应该能够启动执行SomeFunction()的多个线程。我已经对此函数进行了隔离，特别是行anotherInt = pDataArray->val1;。 pdataArray是一个在结构中定义的MyData数组，每个元素都传递给一个线程。

我没有正确分配数组的内存吗？如果没有，我将如何访问作为SomeFunction()参数传入的结构的成员？我已经多次查看我的代码，发现我所知道的任何错误。我在MSDN上遵循的示例是here。

Answer 1

在MyFunction PMYDATA pDataArray;中，pDataArray并未神奇地等同于main中的pDataArray->val1;。它是一个未初始化的指针，LPVOID lparam尝试写入随机内存位置。

提示：您还忽略了{{1}}。

如何在C程序中正确分配内存？

1 个答案: