Mercurial > hg > dmlib
diff tools/ppl.c @ 1787:8af6067b6bd7
Actually it seems to be good to NULL check before calling SDL_Destroy*() and
free functions. According to Valgrind SDL2 "leaks" memory with its error message
setting if you give NULL pointers, although this is pretty harmless.
author | Matti Hamalainen <ccr@tnsp.org> |
---|---|
date | Wed, 13 Jun 2018 01:46:18 +0300 |
parents | 93d1050eac99 |
children | 6147dd3fa5d2 |
line wrap: on
line diff
--- a/tools/ppl.c Wed Jun 13 01:40:22 2018 +0300 +++ b/tools/ppl.c Wed Jun 13 01:46:18 2018 +0300 @@ -910,10 +910,17 @@ } error_exit: - SDL_DestroyTexture(engine.texture); - SDL_DestroyRenderer(engine.renderer); - SDL_DestroyWindow(engine.window); - SDL_FreeSurface(engine.screen); + if (engine.texture != NULL) + SDL_DestroyTexture(engine.texture); + + if (engine.renderer != NULL) + SDL_DestroyRenderer(engine.renderer); + + if (engine.window != NULL) + SDL_DestroyWindow(engine.window); + + if (engine.screen != NULL) + SDL_FreeSurface(engine.screen); dmMsg(0, "Audio shutdown.\n"); if (audioInit)