函数的行为 SDL_FreeSurface:为什么它会在此上下文中崩溃?

Behaviour of function SDL_FreeSurface: why it crashes in this context?

我正在通过在线教程学习SDL,但发现了一个我不明白的问题。 根据本教程,始终正确释放应用程序使用的资源是一种很好的做法。就我而言,我想释放 SDL 表面使用的内存。

但是,如果我在调用该函数之前没有将它将要释放的指针设置为 nullptr,那么 SDL_FreeSurface 会使我的程序在退出时崩溃。

任何其他表面都不会发生这种情况。即使通过更改语句顺序来解决问题,我也不喜欢''cargo cult''。我想了解 WHY 在这种特定情况下我需要使指针 nullptr BEFORE 释放内存。

这是代码:

#define SDL_MAIN_HANDLED
#include<SDL2/SDL.h>

int main (void) {SDL_SetMainReady();

    SDL_Init(SDL_INIT_VIDEO);

    enum image_list{ /* set enum */ };

    SDL_Window  * ptr_Window = nullptr;
    SDL_Surface * ptr_Canvas = nullptr;
    SDL_Surface * ptr_Mask   = nullptr;     // this is the pointer with problem
    SDL_Surface * array_list_of_images[enum_Total]; /* each array initialized correctly */

    ptr_Mask = array_list_of_images[enum_Image_Default]; // here the problematic pointer is set to array[0]

    /* code */

    // Here is where the pointer is used:
    switch (union_events_holder.key.keysym.sym){
        case SDLK_UP    : ptr_Mask = array_list_of_images[enum_Image_Up]; break;
        case SDLK_DOWN  : ptr_Mask = array_list_of_images[enum_Image_Down]; break;
        case SDLK_LEFT  : ptr_Mask = array_list_of_images[enum_Image_Left]; break;
        case SDLK_RIGHT : ptr_Mask = array_list_of_images[enum_Image_Right]; break;
        default         : ptr_Mask = array_list_of_images[enum_Image_Default]; break;

    /* more code */

    SDL_BlitSurface(ptr_Mask , NULL , ptr_Canvas , NULL );
    SDL_UpdateWindowSurface(ptr_Window);

    /* more code */

    // HERE OCCURS THE PROBLEM.
    // WITHOUT POINTING ptr_Mask TO NULL BEFORE CALLING SDL_FreeSurface,
    // THE APP CRASHES.

    ptr_Mask    = nullptr;
    SDL_FreeSurface(ptr_Mask); // This cannot be called before previous statement. Why?

    for ( int ctd = 0 ; ctd < enum_Total ; ++ctd){
        SDL_FreeSurface(array_list_of_images[ctd]);
        array_list_of_images[ctd] = nullptr; // Here it is safe to set pointer to nullptr AFTER freeing memory.
    }   

    SDL_FreeSurface(ptr_Canvas);   
    SDL_DestroyWindow(ptr_Window);

    ptr_Canvas       = nullptr; // Here also it is safe to set pointer to nullptr AFTER freeing memory.
    ptr_Window       = nullptr;

    SDL_Quit();

return (0);}

备注:

编辑 - 这是完整代码(巴西葡萄牙语中的变量)

#define SDL_MAIN_HANDLED
#include<SDL2/SDL.h>

int main (void) {SDL_SetMainReady();

    SDL_Init(SDL_INIT_VIDEO);

    SDL_Window  * ptr_Janela = nullptr;
    SDL_Surface * ptr_Tela   = nullptr;

    enum lista_de_imagens{
        LIMG_Imagem_Default, 
        LIMG_Imagem_Cima,
        LIMG_Imagem_Baixo,
        LIMG_Imagem_Esquerda,
        LIMG_Imagem_Direita,
        LIMG_Total_de_imagens
    };

    SDL_Surface * l_lista_de_imagens[LIMG_Total_de_imagens]; 

        l_lista_de_imagens[LIMG_Imagem_Default]   = SDL_LoadBMP("Lesson4/Default.bmp" );
        l_lista_de_imagens[LIMG_Imagem_Cima]      = SDL_LoadBMP("Lesson4/Cima.bmp"    );
        l_lista_de_imagens[LIMG_Imagem_Baixo]     = SDL_LoadBMP("Lesson4/Baixo.bmp"   );
        l_lista_de_imagens[LIMG_Imagem_Esquerda]  = SDL_LoadBMP("Lesson4/Esquerda.bmp");
        l_lista_de_imagens[LIMG_Imagem_Direita]   = SDL_LoadBMP("Lesson4/Direita.bmp" );
        l_lista_de_imagens[LIMG_Total_de_imagens ]= nullptr;

    SDL_Surface * ptr_mascara = nullptr;

    ptr_Janela = SDL_CreateWindow("Lesson 4 - Show image as keyboard inputs",
                                SDL_WINDOWPOS_UNDEFINED,
                                SDL_WINDOWPOS_UNDEFINED,
                                640, //SCREEN_WIDTH,
                                480, //SCREEN_HEIGHT,
                                SDL_WINDOW_SHOWN);
    ptr_Tela = SDL_GetWindowSurface(ptr_Janela);

    SDL_BlitSurface(l_lista_de_imagens[LIMG_Imagem_Default] , NULL , ptr_Tela , NULL );
    SDL_UpdateWindowSurface(ptr_Janela);

    SDL_Event u_Gerenciador_de_eventos;
    bool sair = false;

    ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Default];

    while(!sair){

            while(SDL_PollEvent(&u_Gerenciador_de_eventos) !=0){

                if(u_Gerenciador_de_eventos.type == SDL_QUIT){sair = true;}
                if( u_Gerenciador_de_eventos.type == SDL_KEYDOWN){

                    switch (u_Gerenciador_de_eventos.key.keysym.sym){

                        case SDLK_UP    : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Cima]; break;

                        case SDLK_DOWN  : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Baixo]; break;

                        case SDLK_LEFT  : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Esquerda]; break;

                        case SDLK_RIGHT : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Direita]; break;

                        default         : ptr_mascara = l_lista_de_imagens[LIMG_Imagem_Default]; break;

                    /*end switch*/}

                /*end if*/}

        /*end laço de eventos*/}

    SDL_BlitSurface(ptr_mascara , NULL , ptr_Tela , NULL );
    SDL_UpdateWindowSurface(ptr_Janela);

    /*end laço principal*/}

    ptr_mascara    = nullptr;
    SDL_FreeSurface(ptr_mascara); 

    for ( int ctd = 0 ; ctd < LIMG_Total_de_imagens ; ++ctd){
        SDL_FreeSurface(l_lista_de_imagens[ctd]);
        l_lista_de_imagens[ctd] = nullptr;
    /*end for*/}    

    SDL_FreeSurface(ptr_Tela);   
    SDL_DestroyWindow(ptr_Janela);

    ptr_Tela       = nullptr;
    ptr_Janela     = nullptr;

    SDL_Quit();

return (0);
/*end main*/}

所选表面被释放两次。

您有两个指针指向同一张图片。第一个在 l_lista_de_imagens[LIMG_Imagem_Default] 中,第二个在 ptr_mascara.

当您执行 SDL_FreeSurface(ptr_mascara) 然后 SDL_FreeSurface(l_lista_de_imagens[ctd]); 时,您将释放同一表面两次。

SDL_FreeSurface(nullptr) 不执行任何操作,因此当您在 SDL_FreeSurface(ptr_mascara) 之前添加 ptr_mascara = nullptr; 时,您只会释放表面一次。