]> granicus.if.org Git - curl/commitdiff
libssh: no need to call sftp_get_error as ssh_get_error is sufficient
authorNikos Mavrogiannopoulos <nmav@gnutls.org>
Sun, 3 Dec 2017 10:58:27 +0000 (11:58 +0100)
committerNikos Mavrogiannopoulos <nmav@gnutls.org>
Sun, 3 Dec 2017 11:34:39 +0000 (12:34 +0100)
Fixes #2141

Signed-off-by: Nikos Mavrogiannopoulos <nmav@gnutls.org>
lib/ssh-libssh.c

index 8856d9044037e61df1a28a1ede96b3e0c2bee638..d138959f3c8ec958dd25b51b89ed6957d9ef1b77 100644 (file)
@@ -896,7 +896,6 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
       if(rc != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
         Curl_safefree(sshc->quote_path2);
-        err = sftp_get_error(sshc->sftp_session);
         failf(data, "Attempt to set SFTP stats failed: %s",
               ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
@@ -916,7 +915,6 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
       if(rc != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
         Curl_safefree(sshc->quote_path2);
-        err = sftp_get_error(sshc->sftp_session);
         failf(data, "symlink command failed: %s",
               ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
@@ -932,7 +930,6 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
                       (mode_t)data->set.new_directory_perms);
       if(rc != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
-        err = sftp_get_error(sshc->sftp_session);
         failf(data, "mkdir command failed: %s",
               ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
@@ -949,7 +946,6 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
       if(rc != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
         Curl_safefree(sshc->quote_path2);
-        err = sftp_get_error(sshc->sftp_session);
         failf(data, "rename command failed: %s",
               ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
@@ -964,7 +960,6 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
       rc = sftp_rmdir(sshc->sftp_session, sshc->quote_path1);
       if(rc != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
-        err = sftp_get_error(sshc->sftp_session);
         failf(data, "rmdir command failed: %s",
               ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
@@ -979,7 +974,6 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
       rc = sftp_unlink(sshc->sftp_session, sshc->quote_path1);
       if(rc != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
-        err = sftp_get_error(sshc->sftp_session);
         failf(data, "rm command failed: %s",
               ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
@@ -997,9 +991,8 @@ static CURLcode myssh_statemach_act(struct connectdata *conn, bool *block)
       statvfs = sftp_statvfs(sshc->sftp_session, sshc->quote_path1);
       if(statvfs != 0 && !sshc->acceptfail) {
         Curl_safefree(sshc->quote_path1);
-        err = sftp_get_error(sshc->sftp_session);
-        failf(data, "statvfs command failed: %s (%d)",
-              ssh_get_error(sshc->ssh_session), err);
+        failf(data, "statvfs command failed: %s",
+              ssh_get_error(sshc->ssh_session));
         state(conn, SSH_SFTP_CLOSE);
         sshc->nextstate = SSH_NO_STATE;
         sshc->actualcode = CURLE_QUOTE_ERROR;