From 74fd585c40d417d917d0248744acd8f7e5cd7c30 Mon Sep 17 00:00:00 2001 From: RickyRister Date: Thu, 26 Jun 2025 09:36:20 -0700 Subject: [PATCH] [PictureLoader] Dumb hack to fix segfault --- .../client/ui/picture_loader/picture_loader_worker_work.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/cockatrice/src/client/ui/picture_loader/picture_loader_worker_work.cpp b/cockatrice/src/client/ui/picture_loader/picture_loader_worker_work.cpp index a92116721..b94772b24 100644 --- a/cockatrice/src/client/ui/picture_loader/picture_loader_worker_work.cpp +++ b/cockatrice/src/client/ui/picture_loader/picture_loader_worker_work.cpp @@ -194,7 +194,10 @@ QImage PictureLoaderWorkerWork::tryLoadImageFromReply(QNetworkReply *reply) void PictureLoaderWorkerWork::concludeImageLoad(const QImage &image) { emit imageLoaded(cardToDownload.getCard(), image); - this->deleteLater(); + + // Delayed delete is a dumb hack to prevent segfaults due to calling methods on a deleted Work + // TODO: find a more proper way to structure this whole thing + QTimer::singleShot(2000, this, &QObject::deleteLater); } void PictureLoaderWorkerWork::picDownloadChanged()