Avoid potential race condtion with QFutureWatcher in FWUPD backend
ClosedPublic

Authored by davidedmundson on Nov 10 2018, 9:34 PM.

Details

Summary

One should connect to the finish slot before starting the threaded job.

Test Plan

Ran
Opened FW updates
Showed none as before, no other issues

Diff Detail

Repository
R134 Discover Software Store
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Nov 10 2018, 9:34 PM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 10 2018, 9:34 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Nov 10 2018, 9:34 PM
apol accepted this revision.Nov 12 2018, 12:39 AM
apol added a subscriber: apol.

Good catch!

This revision is now accepted and ready to land.Nov 12 2018, 12:39 AM
This revision was automatically updated to reflect the committed changes.