derpicated / articated

ARticated; An augmented reality application for Android
MIT License
10 stars 5 forks source link

Leaking Framebuffers #23

Closed Mercotui closed 5 years ago

Mercotui commented 5 years ago

In the function void vision_algorithm::download_image (image_t& image, GLuint handle);, a frame buffer is generated, used and subsequently leaked. This framebuffer should only be generated once, probably at creation of vision_algorithm, and cleaned up at the destruction of vision_algorithm.

Do note that, as the framebuffer handles are 32 bit unsigned integers, so this issue could probably take multiple years of runtime to manifest in undefined behaviour.

Mercotui commented 5 years ago

Fixed in 5d7a97bdf814f45fe383c0e3a4b87558fce4fd9d