Review Request 129993: Fix SFTP unmount

[Yuxuan Shui]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129993/
-----------------------------------------------------------

(Updated March 8, 2017, 1:23 a.m.)


Review request for kdeconnect and Albert Vaca Cintora.


Bugs: 377040
    http://bugs.kde.org/show_bug.cgi?id=377040


Repository: kdeconnect-kde


Description
-------

The destructor of Mounter says it doesn't call unmount because it will crash.

I tracked down the root cause of that. In unmount(), m_proc is set to nullptr before the process is killed. And Stderr/StdoutReadReady is triggered after that. So in the signal handler there's a nullptr dereference.

In this patch I take some actions to make sure the process is shutdown properly.


Diffs (updated)
-----

  plugins/sftp/mounter.h a9edc26 
  plugins/sftp/mounter.cpp f43a445 

Diff: https://git.reviewboard.kde.org/r/129993/diff/


Testing
-------


Thanks,

Yuxuan Shui

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170308/acb5cd91/attachment.html>