[Image layering and Hardware Accelerated Rendering] Възможен SDL_Surface лийк в Тexture::createTextureFromSurface

MartinBG 4803 Точки

[Image layering and Hardware Accelerated Rendering] Възможен SDL_Surface лийк в Тexture::createTextureFromSurface

В Тexture::createTextureFromSurface няма да извикаме freeSurface метода, ако SDL_CreateTextureFromSurface не успее да създаде текстурата.

Фикс:

int32_t Тexture::createTextureFromSurface(SDL_Surface*& inOutSurface, SDL_Texture*& outTexture) {
  outTexture = SDL_CreateTextureFromSurface(gRenderer, inOutSurface);
  freeSurface(inOutSurface); // <- add

  if (!outTexture) {
    logger::log("SDL_CreateTextureFromSurface() failed. Reason: ", SDL_GetError());
    return EXIT_FAILURE;
  }

  //freeSurface(inOutSurface); <- delete

  return EXIT_SUCCESS;
}

Тагове:

09/11/2021 06:27:11 C++ Applications Development

Виж всички отговори

bonev_st 13 Точки

Не мисля, че е необходимо да се освобождава ресурс, който не е създаден. Не съм гледал кода на SDL, но логиката би трябвало да е, ако извикването на функция даде грешка, състоянието да бъде каквото е било преди извикването. В този случай какъв ресурса зад nullptr?

10/11/2021 07:59:50

MartinBG 4803 Точки

В имплементацията от лекциите SDL_Surface пойнтъра (inOutSurface) се създава и подава отвън на createTextureFromSurface, като се очаква да бъде освободен след като от него се създаде текстура.

SDL_CreateTextureFromSurface може да фейлне при създаването на SDL_Texture* (outTexture) и в този случай createTextureFromSurface ще върне EXIT_FAILURE преди да извика freeSurface, който от своя страна извиква SDL_FreeSurface.

10/11/2021 13:38:56 10/11/2021 14:15:05

Виж всички отговори