The-Homebrew-Cloud/dolphin
2
0
Fork 0
mirror of https://github.com/dolphin-emu/dolphin.git synced 2025-01-10 08:09:26 +01:00
Code Issues Packages Projects Releases Wiki Activity

VideoCommon: when loading a PNG with no custom texture data levels already, create a level, this avoids a potential segfault

Browse Source
This commit is contained in:
iwubcode 2023-06-02 22:02:37 -05:00
parent d03e09c8fd
commit 47c40d51df
3 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
Source/Core/VideoCommon/Assets/CustomTextureData.cpp
View File

@ -515,11 +515,6 @@ bool LoadDDSTexture(CustomTextureData::Level* level, const std::string& filename
info.first_mip_row_length, info.first_mip_size); info.first_mip_row_length, info.first_mip_size);
} }
bool LoadPNGTexture(CustomTextureData* texture, const std::string& filename)
{
return LoadPNGTexture(&texture->m_levels[0], filename);
}
bool LoadPNGTexture(CustomTextureData::Level* level, const std::string& filename) bool LoadPNGTexture(CustomTextureData::Level* level, const std::string& filename)
{ {
if (!level) [[unlikely]] if (!level) [[unlikely]]

1
Source/Core/VideoCommon/Assets/CustomTextureData.h
View File

@ -27,6 +27,5 @@ public:
bool LoadDDSTexture(CustomTextureData* texture, const std::string& filename); bool LoadDDSTexture(CustomTextureData* texture, const std::string& filename);
bool LoadDDSTexture(CustomTextureData::Level* level, const std::string& filename, u32 mip_level); bool LoadDDSTexture(CustomTextureData::Level* level, const std::string& filename, u32 mip_level);
bool LoadPNGTexture(CustomTextureData* texture, const std::string& filename);
bool LoadPNGTexture(CustomTextureData::Level* level, const std::string& filename); bool LoadPNGTexture(CustomTextureData::Level* level, const std::string& filename);
} // namespace VideoCommon } // namespace VideoCommon

7
Source/Core/VideoCommon/Assets/DirectFilesystemAssetLibrary.cpp
View File

@ -72,8 +72,11 @@ CustomAssetLibrary::LoadInfo DirectFilesystemAssetLibrary::LoadTexture(const Ass
} }
else if (ext == ".png") else if (ext == ".png")
{ {
LoadPNGTexture(data, asset_path.string()); // If we have no levels, create one to pass into LoadPNGTexture
if (data->m_levels.empty()) [[unlikely]] if (data->m_levels.empty())
data->m_levels.push_back({});
if (!LoadPNGTexture(&data->m_levels[0], asset_path.string()))
return {}; return {};
if (!LoadMips(asset_path, data)) if (!LoadMips(asset_path, data))
return {}; return {};