-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Memory leak with uastc #465
Comments
When is |
Minimal repro: #include <crtdbg.h>
#include "ktx.h"
int main(int, char *[]) {
#ifdef _DEBUG
_CrtSetDbgFlag(_CRTDBG_ALLOC_MEM_DF | _CRTDBG_LEAK_CHECK_DF);
_CrtSetReportMode(_CRT_ASSERT, _CRTDBG_MODE_FILE);
_CrtSetReportFile(_CRT_ASSERT, _CRTDBG_FILE_STDERR);
#endif
ktxTexture *tex;
auto result = ktxTexture_CreateFromNamedFile(
"D:/AO.ktx", KTX_TEXTURE_CREATE_LOAD_IMAGE_DATA_BIT, &tex);
ktxTexture_Destroy(tex);
return 0;
} https://drive.google.com/file/d/1Op0nDvdIfW6tJP5XVuSuHnZ3JvNenrza/view?usp=sharing |
Which version of the software are you using? On Mac the allocator diagnostics are showing any leaks with the latest version of the software. The example in your most recent comment does not show a call to ktxTexture2_TranscodeBasis. Is that intentional? The texture you posted has |
Yes, it looks like That code is actual minimal reproduction for the memory leak - just create and destroy. |
Lastest version (master)
https://drive.google.com/file/d/1Op0nDvdIfW6tJP5XVuSuHnZ3JvNenrza/view?usp=sharing
Image converted with
toktx
--uastc 4 --uastc_rdo_l .5 --uastc_rdo_d 65536 --zcmp 22
--bcmp
is ok, no leaksThe text was updated successfully, but these errors were encountered: