<table><tr><td style="">johngehrig updated this revision to Diff 57581.<br />johngehrig edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-6fwtuhaxns72vk2/">(Show Details)</a><br />johngehrig edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-v7egd7gtumfmwrw/">(Show Details)</a><br />johngehrig added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D20859">View Revision</a></tr></table><br /><div><div><p>Ordering changed, qWarning first.<br />
job->errorString() displayed for all values of job->error().<br />
A QString object was created, so job is no longer passed to KMessageBox.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">The <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">function "KJob::errorString()" should only be called when "error() != 0". The current call pattern does not adequately prevent this. Crashes have been observed when "!rc && error() == 0"</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">destruction of KAuth::ExecuteJob* "job" is not handled properly when passed as a parameter to KMessageBox</span>.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Checks for this case have been added with error an message of "unknown error"</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">An intermediate QString is created containing the error text. This value is passed into the KMessageBox and qWarning() so that the object "job" is no longer an input. The console error is now first so an error message displayed even in the event of a GUI crash</span>.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">I have applied this patch</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">The patch has been applied</span> to my system<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">,</span> and the crash no longer occurs.</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20859?vs=57081&id=57581">https://phabricator.kde.org/D20859?vs=57081&id=57581</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20859">https://phabricator.kde.org/D20859</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>sddmthemeinstaller.cpp</div></div></div><br /><div><strong>To: </strong>johngehrig, ngraham, davidedmundson, Plasma<br /><strong>Cc: </strong>anthonyfieroni, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>