BaseJob: don't finish and re-try at the same time

Due to a missing return statement, a retry with auth case led to the job
being finished and pending at the same time, with no good consequences.

1 files changed, 3 insertions, 0 deletions

diff --git a/lib/jobs/basejob.cpp b/lib/jobs/basejob.cpp
index dfc3d3dd..7cddc343 100644
--- a/lib/jobs/basejob.cpp
+++ b/lib/jobs/basejob.cpp
@@ -527,6 +527,7 @@ void BaseJob::finishJob()
         qCWarning(d->logCat) << this << "re-running with authentication";
         emit retryScheduled(d->retriesTaken, 0);
         d->connection->submit(this);
+        return;
     }
     if ((error() == NetworkError || error() == Timeout)
         && d->retriesTaken < d->maxRetries) {
@@ -542,6 +543,8 @@ void BaseJob::finishJob()
         return;
     }
 
+    Q_ASSERT(status().code != Pending);
+
     // Notify those interested in any completion of the job including abandon()
     emit finished(this);