https://bugs.kde.org/show_bug.cgi?id=404609

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |5.16.0
      Latest Commit|                            |https://commits.kde.org/sdd
                   |                            |m-kcm/7df456404c26200ff6e36
                   |                            |fc3d65aa39f2c6e7c3c
         Resolution|---                         |FIXED
             Status|REPORTED                    |RESOLVED

--- Comment #5 from Nate Graham <n...@kde.org> ---
Git commit 7df456404c26200ff6e36fc3d65aa39f2c6e7c3c by Nate Graham, on behalf
of John Gehrig.
Committed on 14/05/2019 at 20:37.
Pushed by ngraham into branch 'master'.

Crash in sddmthemeinstaller invalid use of errorString

Summary:
The destruction of KAuth::ExecuteJob* "job" is not handled properly when passed
as a parameter to KMessageBox.

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.
Related: bug 406718
FIXED-IN: 5.16.0

Test Plan: The patch has been applied to my system and the crash no longer
occurs.

Reviewers: ngraham, davidedmundson, #plasma

Reviewed By: ngraham, davidedmundson, #plasma

Subscribers: anthonyfieroni, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D20859

M  +3    -2    sddmthemeinstaller.cpp

https://commits.kde.org/sddm-kcm/7df456404c26200ff6e36fc3d65aa39f2c6e7c3c

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to