Fix crash that could occur when closing down splits (#4277)

* Fix crash that could occur when closing down splits

Specifically, if a split was closed after the request for a thumbnail
had been made, but before the request had finished, we'd run into a
use-after-free issue

* Add changelog entry
This commit is contained in:
pajlada 2022-12-31 13:39:28 +01:00 committed by GitHub
parent 5fc170ba4b
commit 293aa370c5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 0 deletions

View file

@ -18,6 +18,7 @@
- Bugfix: Fixed text sometimes not pasting properly when image uploader was disabled. (#4246)
- Bugfix: Fixed text cursor(caret) not showing in open channel dialog. (#4196)
- Bugfix: Fixed tooltip images not appearing if mouse hovered only first pixel. (#4268)
- Bugfix: Fixed crash that could occurr when closing down a split at the wrong time. (#4277)
- Dev: Remove protocol from QApplication's Organization Domain (so changed from `https://www.chatterino.com` to `chatterino.com`). (#4256)
- Dev: Ignore `WM_SHOWWINDOW` hide events, causing fewer attempted rescales. (#4198)
- Dev: Migrated to C++ 20 (#4252, #4257)

View file

@ -804,6 +804,7 @@ void SplitHeader::updateChannelText()
this->lastThumbnail_.elapsed() > THUMBNAIL_MAX_AGE_MS))
{
NetworkRequest(url, NetworkRequestType::Get)
.caller(this)
.onSuccess([this](auto result) -> Outcome {
// NOTE: We do not follow the redirects, so we need to make sure we only treat code 200 as a valid image
if (result.status() == 200)