We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Native win32 test case for possibly related problem.

win32_crash_leak_investigation.cpp (text/plain), 4.31 KB, created by Filippo Crocchini on 2020-12-31 11:32:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Filippo Crocchini

Created: 2020-12-31 11:32:48 UTC

Size: 4.31 KB

patch

obsolete

>#include <Windows.h>
>#include <GL\GL.h>
>
>// Almost everything here was copied from here http://archive.gamedev.net/archive/reference/articles/article540.html
>
>void
>display()
>{
>    glClear(GL_COLOR_BUFFER_BIT);
>    glBegin(GL_TRIANGLES);
>    glColor3f(1.0f, 0.0f, 0.0f);
>    glVertex2i(0, 1);
>    glColor3f(0.0f, 1.0f, 0.0f);
>    glVertex2i(-1, -1);
>    glColor3f(0.0f, 0.0f, 1.0f);
>    glVertex2i(1, -1);
>    glEnd();
>    glFlush();
>}
>
>HDC   hDC;
>
>LONG WINAPI
>WindowProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
>{
>    static PAINTSTRUCT ps;
>    switch (uMsg) {
>    case WM_PAINT: {
>        display();
>        wglSwapLayerBuffers(hDC, WGL_SWAP_MAIN_PLANE);
>        // I also tried SwapBuffers(hDC) which, after some resizing, crashes inside the drivers.
>
>        // Both SwapBuffers and wglSwapLayerBuffers leak memory on resize, although on Intel they leak way more while
>        // on Nvidia it looks like it caps at around 600MB and it seems to decrease when the size is it's not decreased.
>        // It almost behaves like they have some garbage collection going on-
>
>        ValidateRect(hWnd, NULL); // This is only for the WM_PAINT messages that didn't come from the WM_SIZE
>        return 0;
>    }
>    case WM_SIZE:
>        glViewport(0, 0, LOWORD(lParam), HIWORD(lParam));
>
>        // I'm doing this to force a WM_PAINT every time the window is resized, otherwise it would only happen sometimes
>        // It looks like only when making the window bigger for some reason.
>        PostMessageA(hWnd, WM_PAINT, 0, 0);
>        ValidateRect(hWnd, NULL); // This is to avoid double WM_PAINT messages
>        return 0;
>
>    case WM_CLOSE:
>    case WM_DESTROY:
>        PostQuitMessage(0);
>        return 0;
>    }
>
>    return DefWindowProc(hWnd, uMsg, wParam, lParam);
>}
>
>HWND
>CreateOpenGLWindow(char* title, int x, int y, int width, int height)
>{
>    HWND        hWnd;
>    WNDCLASS    wc;
>    static HINSTANCE hInstance = 0;
>
>    if (!hInstance) {
>        hInstance = GetModuleHandle(NULL);
>        wc.style = CS_OWNDC;
>        wc.lpfnWndProc = (WNDPROC)WindowProc;
>        wc.cbClsExtra = 0;
>        wc.cbWndExtra = 0;
>        wc.hInstance = hInstance;
>        wc.hIcon = LoadIcon(NULL, IDI_WINLOGO);
>        wc.hCursor = LoadCursor(NULL, IDC_ARROW);
>        wc.hbrBackground = NULL;
>        wc.lpszMenuName = NULL;
>        wc.lpszClassName = "OpenGL";
>
>        if (!RegisterClass(&wc)) {
>            MessageBox(NULL, "RegisterClass() failed:  "
>                "Cannot register window class.", "Error", MB_OK);
>            return NULL;
>        }
>    }
>
>    hWnd = CreateWindow("OpenGL", title, WS_OVERLAPPEDWINDOW |
>        WS_CLIPSIBLINGS | WS_CLIPCHILDREN,
>        x, y, width, height, NULL, NULL, hInstance, NULL);
>
>    if (hWnd == NULL) {
>        MessageBox(NULL, "CreateWindow() failed:  Cannot create a window.",
>            "Error", MB_OK);
>        return NULL;
>    }
>    return hWnd;
>}
>
>
>int APIENTRY
>WinMain(HINSTANCE hCurrentInst, HINSTANCE hPreviousInst,
>    LPSTR lpszCmdLine, int nCmdShow)
>{
>    HGLRC hRC; 
>    HWND  hWnd;
>
>    hWnd = CreateOpenGLWindow("minimal", 0, 0, 256, 256);
>    if (hWnd == NULL)
>        exit(1);
>
>    hDC = GetDC(hWnd);
>
>    PIXELFORMATDESCRIPTOR pfd = {};
>    pfd.nSize = sizeof(pfd);
>    pfd.nVersion = 1;
>    pfd.dwFlags = PFD_DRAW_TO_WINDOW | PFD_SUPPORT_OPENGL | PFD_DOUBLEBUFFER;
>    pfd.iPixelType = PFD_TYPE_RGBA;
>    pfd.cColorBits = 32;
>
>    int pf = ChoosePixelFormat(hDC, &pfd);
>    if (pf == 0) {
>        MessageBox(NULL, "ChoosePixelFormat() failed:  "
>            "Cannot find a suitable pixel format.", "Error", MB_OK);
>        return 0;
>    }
>
>    DescribePixelFormat(hDC, pf, sizeof(PIXELFORMATDESCRIPTOR), &pfd);
>
>    if (SetPixelFormat(hDC, pf, &pfd) == FALSE) {
>        MessageBox(NULL, "SetPixelFormat() failed:  "
>            "Cannot set format specified.", "Error", MB_OK);
>        return 0;
>    }
>
>
>    hRC = wglCreateContext(hDC);
>    wglMakeCurrent(hDC, hRC);
>
>    ShowWindow(hWnd, nCmdShow);
>    UpdateWindow(hWnd);
>
>    MSG   msg;
>    while (GetMessageW(&msg, hWnd, 0, 0)) {
>        TranslateMessage(&msg);
>        DispatchMessageW(&msg);
>    }
>
>    wglMakeCurrent(hDC, NULL);
>    wglDeleteContext(hRC);
>    ReleaseDC(hWnd, hDC);
>    DestroyWindow(hWnd);
>
>    return msg.wParam;
>}

Actions: View

Attachments on bug 5423: 4622

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.