From bd14854f1d4790aeea5e39ee6ceabbd690331761 Mon Sep 17 00:00:00 2001
From: Kitsune Ral <Kitsune-Ral@users.sf.net>
Date: Mon, 30 Mar 2020 19:12:06 +0200
Subject: BaseJob: shutdown timers on abandoning and destruction

A part of the fix for #398.
---
 lib/jobs/basejob.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/lib/jobs/basejob.cpp b/lib/jobs/basejob.cpp
index a1f49b63..30d60f9e 100644
--- a/lib/jobs/basejob.cpp
+++ b/lib/jobs/basejob.cpp
@@ -104,6 +104,7 @@ BaseJob::BaseJob(HttpVerb verb, const QString& name, const QString& endpoint,
 BaseJob::~BaseJob()
 {
     stop();
+    d->retryTimer.stop(); // See #398
     qCDebug(d->logCat) << this << "destroyed";
 }
 
@@ -641,6 +642,8 @@ void BaseJob::setStatus(int code, QString message)
 void BaseJob::abandon()
 {
     beforeAbandon(d->reply ? d->reply.data() : nullptr);
+    d->timer.stop();
+    d->retryTimer.stop(); // In case abandon() was called between retries
     setStatus(Abandoned);
     if (d->reply)
         d->reply->disconnect(this);
-- 
cgit v1.2.3