Skip to content

Commit

Permalink
Leak mutex to avoid a crashing issue
Browse files Browse the repository at this point in the history
  • Loading branch information
chuganzy committed Dec 11, 2018
1 parent 39b6890 commit 7d55b4e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions React/Views/RCTFont.mm
Original file line number Diff line number Diff line change
Expand Up @@ -126,12 +126,12 @@ static inline BOOL CompareFontWeights(UIFontWeight firstWeight, UIFontWeight sec
static UIFont *cachedSystemFont(CGFloat size, RCTFontWeight weight)
{
static NSCache *fontCache;
static std::mutex fontCacheMutex;
static std::mutex *fontCacheMutex = new std::mutex;

NSString *cacheKey = [NSString stringWithFormat:@"%.1f/%.2f", size, weight];
UIFont *font;
{
std::lock_guard<std::mutex> lock(fontCacheMutex);
std::lock_guard<std::mutex> lock(*fontCacheMutex);
if (!fontCache) {
fontCache = [NSCache new];
}
Expand All @@ -158,7 +158,7 @@ static inline BOOL CompareFontWeights(UIFontWeight firstWeight, UIFontWeight sec
}

{
std::lock_guard<std::mutex> lock(fontCacheMutex);
std::lock_guard<std::mutex> lock(*fontCacheMutex);
[fontCache setObject:font forKey:cacheKey];
}
}
Expand Down

0 comments on commit 7d55b4e

Please sign in to comment.