函数的行为 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);}
备注:
标记为 C 因为 SDL。我正在用 C++ 编码。
通过 std::cerr
+ std::cin.get()
逐步找到错误
SDL_GetError()
无法输出信息,因为Window和控制台在调用该函数时都崩溃了。
我怀疑取消引用 nullptr
的未定义行为,但这没有任何意义,一旦函数仅在我设置为 nullptr
.
时才正常工作
编辑 - 这是完整代码(巴西葡萄牙语中的变量)
#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;
时,您只会释放表面一次。
我正在通过在线教程学习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);}
备注:
标记为 C 因为 SDL。我正在用 C++ 编码。
通过
std::cerr
+std::cin.get()
逐步找到错误
SDL_GetError()
无法输出信息,因为Window和控制台在调用该函数时都崩溃了。我怀疑取消引用
时才正常工作nullptr
的未定义行为,但这没有任何意义,一旦函数仅在我设置为nullptr
.
编辑 - 这是完整代码(巴西葡萄牙语中的变量)
#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;
时,您只会释放表面一次。