git_clone/thorvg
1
0
Fork 0
mirror of https://github.com/thorvg/thorvg.git synced 2025-06-08 13:43:43 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

lottie: fix a data-race problem

Browse source 
make it sync call before accessing the composition data
from marker and segments.

issue: https://github.com/thorvg/thorvg/issues/2462
This commit is contained in:
Hermet Park 2024-06-27 01:01:07 +09:00 • committed by Hermet Park
parent b2def251ca
commit 87cd9e4795
1 changed files with 9 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/loaders/lottie/tvgLottieLoader.cpp
View file

@ -273,7 +273,7 @@ Paint* LottieLoader::paint()
bool LottieLoader::override(const char* slot)
{
if (!comp) done();
done();
if (!comp || comp->slots.count == 0) return false;
@ -365,7 +365,7 @@ float LottieLoader::duration()
void LottieLoader::sync()
{
this->done();
done();
if (rebuild) run(0);
}
@ -373,15 +373,15 @@ void LottieLoader::sync()
uint32_t LottieLoader::markersCnt()
{
if (!comp) done();
if (!comp) return 0;
return comp->markers.count;
done();
return comp ? comp->markers.count : 0;
}
const char* LottieLoader::markers(uint32_t index)
{
if (!comp) done();
done();
if (!comp || index >= comp->markers.count) return nullptr;
auto marker = comp->markers.begin() + index;
return (*marker)->name;
@ -390,7 +390,8 @@ const char* LottieLoader::markers(uint32_t index)
bool LottieLoader::segment(const char* marker, float& begin, float& end)
{
if (!comp) done();
done();
if (!comp) return false;
for (auto m = comp->markers.begin(); m < comp->markers.end(); ++m) {