https://bugs.kde.org/show_bug.cgi?id=365054
Matthew Dawson <matt...@mjdsystems.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED Latest Commit| |http://commits.kde.org/krdc | |/a6dca605fae01440cb510d65fd | |1896ba77646722 --- Comment #5 from Matthew Dawson <matt...@mjdsystems.ca> --- Git commit a6dca605fae01440cb510d65fd1896ba77646722 by Matthew Dawson. Committed on 01/08/2016 at 17:24. Pushed by mdawson into branch 'Applications/16.08'. Avoid crash when handling RDP error and xfreerdp quits. If xfreerdp sent a failure message then quit, the RDP plugin would first show a message box with a relevant message. During that time, the inner event loop would handle the xfreerdp process quitting, which would signal krdc to cleanup the RDP plugin's resources. When the message box event loop completed, the RdpView would have been destroyed, causing a crash. Now the RdpView signals that the view is quitting as soon as the message box is shown, and avoids trying to quit twice if another part signals a quit. Once the message box is dismissed, the process continues as normal. This adds the necessary mechanisms to handle this, as well as moving the message box display code to connectionError, to avoid some duplication. REVIEW: 128569 M +24 -20 rdp/rdpview.cpp M +10 -1 rdp/rdpview.h http://commits.kde.org/krdc/a6dca605fae01440cb510d65fd1896ba77646722 -- You are receiving this mail because: You are watching all bug changes.