ShuriZma
/
dolphin
mirror of https://github.com/dolphin-emu/dolphin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

TextureCache: Fix leaking TCacheEntry instances

This commit is contained in:
Stenzek 2018-11-08 11:52:17 +10:00
parent 16a618f413
commit 78056686fd
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Source/Core/VideoCommon/TextureCacheBase.cpp
View File

@ -1923,13 +1923,12 @@ void TextureCacheBase::FlushEFBCopy(TCacheEntry* entry)
WriteEFBCopyToRAM(dst, entry->pending_efb_copy_width, entry->pending_efb_copy_height, WriteEFBCopyToRAM(dst, entry->pending_efb_copy_width, entry->pending_efb_copy_height,
entry->memory_stride, std::move(entry->pending_efb_copy)); entry->memory_stride, std::move(entry->pending_efb_copy));
// If the EFB copy was invalidated (e.g. the bloom case mentioned in InvalidateTexture), // If the EFB copy was invalidated (e.g. the bloom case mentioned in InvalidateTexture), now is
// now is the time to clean up the TCacheEntry. In which case, we don't need to compute // the time to clean up the TCacheEntry. In which case, we don't need to compute the new hash of
// the new hash of the RAM copy. // the RAM copy. But we need to clean up the TCacheEntry, as InvalidateTexture doesn't free it.
if (entry->pending_efb_copy_invalidated) if (entry->pending_efb_copy_invalidated)
{ {
auto config = entry->texture->GetConfig(); delete entry;
texture_pool.emplace(config, TexPoolEntry(std::move(entry->texture)));
return; return;
} }
@ -2137,13 +2136,16 @@ TextureCacheBase::InvalidateTexture(TexAddrCache::iterator iter, bool discard_pe
else else
{ {
entry->pending_efb_copy_invalidated = true; entry->pending_efb_copy_invalidated = true;
return textures_by_address.erase(iter);
} }
} }
auto config = entry->texture->GetConfig(); auto config = entry->texture->GetConfig();
texture_pool.emplace(config, TexPoolEntry(std::move(entry->texture))); texture_pool.emplace(config, TexPoolEntry(std::move(entry->texture)));
// Don't delete if there's a pending EFB copy, as we need the TCacheEntry alive.
if (!entry->pending_efb_copy)
delete entry;
return textures_by_address.erase(iter); return textures_by_address.erase(iter);
} }