SSH error messages not shown

From: Jacek Wielemborek <d33tah(at)gmail(dot)com>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: SSH error messages not shown
Date: 2015-06-13 15:22:10
Message-ID: 557C4AA2.3000401@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

When an libssh2 call fails, pgAdmin III just shows the error code and
nothing else. Consider the following patch - this is what I created
while wondering why my hostkeys don't work:

diff --git a/pgadmin/utils/sshTunnel.cpp b/pgadmin/utils/sshTunnel.cpp
index fd80214..e15f764 100644
--- a/pgadmin/utils/sshTunnel.cpp
+++ b/pgadmin/utils/sshTunnel.cpp
@@ -201,8 +201,13 @@ bool CSSHTunnelThread::Initialize()
#endif
if (rc)
{
-
LogSSHTunnelErrors(wxString::Format(_("SSH error: Authentication by
identity file failed with error code %d"), rc), GetId());
+ char* errmsg;
+ int errmsg_len;
+ libssh2_session_last_error(m_session,
&errmsg, &errmsg_len, 0);
+ wxString errmsg_s(errmsg, wxConvLibc);
+
LogSSHTunnelErrors(wxString::Format(_("SSH error: Authentication by
identity file failed with error code %d [%s]"), rc, errmsg_s.c_str()),
GetId());
Cleanup();
+ free(errmsg);
return false;
}
}

Perhaps we could use this kind of routines wherever rc is being checked?

Cheers,
d33tah

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message John Obaterspok 2015-06-14 08:30:16 Re: [PATCH] Fix crash when disabling auto commit
Previous Message Jacek Wielemborek 2015-06-13 15:20:15 valgrind complains a lot when trying to establish an SSH tunnel while giving wrong key