diff options
| author | Christian Kandeler <christian.kandeler@nokia.com> | 2011-11-09 17:38:19 +0100 |
|---|---|---|
| committer | Christian Kandeler <christian.kandeler@nokia.com> | 2011-11-09 17:41:27 +0100 |
| commit | 5d5db225a82964a40178abc360811456dabda8a4 (patch) | |
| tree | ba3933d2244674c140000ee3617da378a5d44f99 /src/plugins/madde/maddedevicetester.cpp | |
| parent | bddfae718242d8bf2b6868cb6628f97c9b3a10f2 (diff) | |
| download | qt-creator-5d5db225a82964a40178abc360811456dabda8a4.tar.gz | |
SSH: Don't allow external access to process runner's connection object.
It does not belong to the runner anymore after the process has finished,
so we must prevent anyone from getting a copy.
(This includes a minor API fix for RemoteLinux code that depended on
the connection being available.)
Change-Id: Icc28dc3c820f4c2210bb6fb08171cd6a84534dee
Reviewed-by: Christian Kandeler <christian.kandeler@nokia.com>
Diffstat (limited to 'src/plugins/madde/maddedevicetester.cpp')
| -rw-r--r-- | src/plugins/madde/maddedevicetester.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/plugins/madde/maddedevicetester.cpp b/src/plugins/madde/maddedevicetester.cpp index 2ca4d09425..658693de4c 100644 --- a/src/plugins/madde/maddedevicetester.cpp +++ b/src/plugins/madde/maddedevicetester.cpp @@ -109,8 +109,7 @@ void MaddeDeviceTester::handleGenericTestFinished(TestResult result) if (!m_processRunner) m_processRunner = new SshRemoteProcessRunner(this); - connect(m_processRunner, SIGNAL(connectionError(Utils::SshError)), - SLOT(handleConnectionError())); + connect(m_processRunner, SIGNAL(connectionError()), SLOT(handleConnectionError())); connect(m_processRunner, SIGNAL(processOutputAvailable(QByteArray)), SLOT(handleStdout(QByteArray))); connect(m_processRunner, SIGNAL(processErrorOutputAvailable(QByteArray)), @@ -137,7 +136,7 @@ void MaddeDeviceTester::handleConnectionError() QTC_ASSERT(m_state != Inactive, return); emit errorMessage(tr("SSH connection error: %1\n") - .arg(m_processRunner->connection()->errorString())); + .arg(m_processRunner->lastConnectionErrorString())); m_result = TestFailure; setFinished(); } |