From e63cbd907f146a3ad3b435f1ccaab765033b80e9 Mon Sep 17 00:00:00 2001
From: Daniel Stenberg <daniel@haxx.se>
Date: Mon, 6 Apr 2009 14:50:29 +0000
Subject: [PATCH] Jussi Mononen pointed out we used an unsigned variable to
 store negative values in, when a plain int is much better anyway since it
 matches the return type.

---
 src/sftp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/sftp.c b/src/sftp.c
index ee3a9cb..e17b2e9 100644
--- a/src/sftp.c
+++ b/src/sftp.c
@@ -1894,7 +1894,8 @@ static int sftp_rename(LIBSSH2_SFTP *sftp, const char *source_filename,
 {
     LIBSSH2_CHANNEL *channel = sftp->channel;
     LIBSSH2_SESSION *session = channel->session;
-    unsigned long data_len, retcode;
+    unsigned long data_len;
+    int retcode;
     ssize_t packet_len =
         source_filename_len + dest_filename_len + 17 + (sftp->version >=
                                                         5 ? 4 : 0);