调用 GetDIBits() 成功，但程序终止

Question

当我在 Windows 程序中调用以下函数时，程序突然终止。

ScanRect()的目的是在屏幕上的指定坐标处复制一个矩形，并将像素值加载到内存缓冲区中。

ScanRect() 中的每个函数调用都会成功，包括对GetDIBits() 的两次调用。第一次调用，将lpvBits 设置为NULL，使它用有关像素数据的信息填充bmInfo 的BITMAPINFOHEADER，报告每像素32 位的值。第二次调用GetDIBits() 将矩形的80 行复制到内存缓冲区pMem，返回值80 表示复制的行数。

一切似乎都成功了，但随后程序突然终止。我在第二次调用GetDIBits() 后插入了Sleep(8192) 行，程序在8 秒后终止。

是什么导致程序终止？

编辑：原始代码根据此线程中的建议进行了修改。函数运行时没有检测到错误，但程序仍然意外终止。我意识到内存缓冲区大小是硬编码的，但它比测试中使用的矩形所需的要大得多。这不应该导致错误。当然，在找出程序终止的原因后，我会让程序计算必要的缓冲区大小。

VOID ScanRect(int x, int y, int iWidth, int iHeight) // 992, 96, 64, 80
{   HDC hDC = GetDC(NULL);
    if (!hDC)
    {
      cout << "!hDC" << endl;        // error handling ...
    }
    else
    {   HBITMAP hBitmap = CreateCompatibleBitmap(hDC, iWidth, iHeight);
        if (!hBitmap)
        {
           cout << "!hBitmap" << endl;        // error handling ...
        }
        else
        {   HDC hCDC = CreateCompatibleDC(hDC); // compatible with screen DC
            if (!hCDC)
            {
              cout << "!hCDC" << endl;        // error handling ...
            }
            else
            {   HBITMAP hOldBitmap = (HBITMAP) SelectObject(hCDC, hBitmap);
                BitBlt(hCDC, 0, 0, iWidth, iHeight, hDC, x, y, SRCCOPY);
                BITMAPINFO bmInfo = {0};
                bmInfo.bmiHeader.biSize = sizeof(bmInfo.bmiHeader);
                if (!GetDIBits(hCDC, hBitmap, 0, iHeight, NULL, &bmInfo, DIB_RGB_COLORS))
                {
                  cout << "!GetDIBits" << endl; // error handling ...
                }
                else
                {   HANDLE hHeap = GetProcessHeap();
                    LPVOID pMem = HeapAlloc(hHeap, HEAP_ZERO_MEMORY, 65536); // TODO: calculate a proper size based on bmInfo's pixel information ...
                    if (!pMem)
                    {
                      cout << "!pMem" << endl;
                    }
                    else
                    {   int i = GetDIBits(hCDC, hBitmap, 0, iHeight, pMem, &bmInfo, DIB_RGB_COLORS);
                        cout << "i returned by GetDIBits() " << i << endl;
                        HeapFree(hHeap, NULL, pMem);
                    }
                }
                SelectObject(hCDC, hOldBitmap);
                DeleteDC(hCDC);
            }
            DeleteObject(hBitmap);
        }
        ReleaseDC(NULL, hDC);
    }
}

Answer 1

biCompression 值是由第一个 GetDIBits 返回的 BI_BITFIELDS 并且在您调用第二个 GetDIBits 之前，您需要调用 bmInfo.bmiHeader.biCompression = BI_RGB;。根据c++ read pixels with GetDIBits()，将其设置为BI_RGB 是必不可少的，以避免在结构末尾写入额外的3 个DWORD。
More details

Answer 2

就像@BenVoigt 在 cmets 中所说，在销毁拥有它的 HDC 之前，您需要恢复用 SelectObject() 替换的旧 HBITMAP。您将hBitmap 选择为hCDC，然后在销毁hBitmap 之前销毁hCDC。

https://docs.microsoft.com/en-us/windows/win32/gdi/operations-on-graphic-objects

这些函数中的每一个都返回一个标识新对象的句柄。在应用程序检索到句柄后，它必须调用SelectObject() 函数来替换默认对象。 但是，应用程序应保存标识默认对象的句柄，并在不再需要时使用此句柄替换新对象。当应用程序使用新对象完成绘制后，必须调用SelectObject() 函数恢复默认对象，然后调用DeleteObject() 函数删除新对象。 无法删除对象会导致严重的性能问题。

此外，您应该按照创建它们的相反顺序释放 GDI 对象。

而且，不要忘记错误处理。

试试类似的方法：

VOID ScanRect(int x, int y, int iWidth, int iHeight) // 992, 96, 64, 80
{ 
    HDC hDC = GetDC(NULL);
    if (!hDC)
    {
        // error handling ...
    }
    else
    {
        HBITMAP hBitmap = CreateCompatibleBitmap(hDC, iWidth, iHeight);
        if (!hBitmap)
        {
            // error handling ...
        }
        else
        {
            HDC hCDC = CreateCompatibleDC(hDC); // compatible with screen DC
            if (!hCDC)
            {
                // error handling ...
            }
            else
            {
                HBITMAP hOldBitmap = (HBITMAP) SelectObject(hCDC, hBitmap);

                BitBlt(hCDC, 0, 0, iWidth, iHeight, hDC, x, y, SRCCOPY);

                SelectObject(hCDC, hOldBitmap);

                BITMAPINFO bmInfo = {0};
                bmInfo.bmiHeader.biSize = sizeof(bmInfo.bmiHeader);

                if (!GetDIBits(hCDC, hBitmap, 0, iHeight, NULL, &bmInfo, DIB_RGB_COLORS))
                {
                    // error handling ...
                }
                else
                {
                    HANDLE hHeap = GetProcessHeap();
                    LPVOID pMem = HeapAlloc(hHeap, HEAP_ZERO_MEMORY, 65536); // TODO: calculate a proper size based on bmInfo's pixel information ...
                    if (!pMem)
                    {
                        // error handling ...
                    }
                    else
                    {
                        int i = GetDIBits(hCDC, hBitmap, 0, iHeight, pMem, &bmInfo, DIB_RGB_COLORS);
                        HeapFree(hHeap, NULL, pMem);
                    }
                }

                DeleteDC(hCDC);
            }

            DeleteObject(hBitmap);
        }

        ReleaseDC(NULL, hDC);
    }
}

注意对HeapAlloc() 的调用中的TODO。你真的应该根据位图的实际宽度、高度、像素深度、扫描线填充大小等来计算缓冲区大小。不要使用硬编码的缓冲区大小。我将把它作为一个练习让你弄清楚。虽然在这个特定的例子中，64K 对于 64x80 32bpp 位图来说应该足够大，但它只会浪费 45K 的未使用内存。