diff options
Diffstat (limited to 'sftp-server.c')
-rw-r--r-- | sftp-server.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/sftp-server.c b/sftp-server.c index e82280057..6870e7732 100644 --- a/sftp-server.c +++ b/sftp-server.c | |||
@@ -14,7 +14,7 @@ | |||
14 | * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. | 14 | * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. |
15 | */ | 15 | */ |
16 | #include "includes.h" | 16 | #include "includes.h" |
17 | RCSID("$OpenBSD: sftp-server.c,v 1.47 2004/06/25 05:38:48 dtucker Exp $"); | 17 | RCSID("$OpenBSD: sftp-server.c,v 1.48 2005/06/17 02:44:33 djm Exp $"); |
18 | 18 | ||
19 | #include "buffer.h" | 19 | #include "buffer.h" |
20 | #include "bufaux.h" | 20 | #include "bufaux.h" |
@@ -130,7 +130,7 @@ Handle handles[100]; | |||
130 | static void | 130 | static void |
131 | handle_init(void) | 131 | handle_init(void) |
132 | { | 132 | { |
133 | int i; | 133 | u_int i; |
134 | 134 | ||
135 | for (i = 0; i < sizeof(handles)/sizeof(Handle); i++) | 135 | for (i = 0; i < sizeof(handles)/sizeof(Handle); i++) |
136 | handles[i].use = HANDLE_UNUSED; | 136 | handles[i].use = HANDLE_UNUSED; |
@@ -139,7 +139,7 @@ handle_init(void) | |||
139 | static int | 139 | static int |
140 | handle_new(int use, const char *name, int fd, DIR *dirp) | 140 | handle_new(int use, const char *name, int fd, DIR *dirp) |
141 | { | 141 | { |
142 | int i; | 142 | u_int i; |
143 | 143 | ||
144 | for (i = 0; i < sizeof(handles)/sizeof(Handle); i++) { | 144 | for (i = 0; i < sizeof(handles)/sizeof(Handle); i++) { |
145 | if (handles[i].use == HANDLE_UNUSED) { | 145 | if (handles[i].use == HANDLE_UNUSED) { |
@@ -156,7 +156,7 @@ handle_new(int use, const char *name, int fd, DIR *dirp) | |||
156 | static int | 156 | static int |
157 | handle_is_ok(int i, int type) | 157 | handle_is_ok(int i, int type) |
158 | { | 158 | { |
159 | return i >= 0 && i < sizeof(handles)/sizeof(Handle) && | 159 | return i >= 0 && (u_int)i < sizeof(handles)/sizeof(Handle) && |
160 | handles[i].use == type; | 160 | handles[i].use == type; |
161 | } | 161 | } |
162 | 162 | ||
@@ -477,10 +477,10 @@ process_write(void) | |||
477 | } else { | 477 | } else { |
478 | /* XXX ATOMICIO ? */ | 478 | /* XXX ATOMICIO ? */ |
479 | ret = write(fd, data, len); | 479 | ret = write(fd, data, len); |
480 | if (ret == -1) { | 480 | if (ret < 0) { |
481 | error("process_write: write failed"); | 481 | error("process_write: write failed"); |
482 | status = errno_to_portable(errno); | 482 | status = errno_to_portable(errno); |
483 | } else if (ret == len) { | 483 | } else if ((size_t)ret == len) { |
484 | status = SSH2_FX_OK; | 484 | status = SSH2_FX_OK; |
485 | } else { | 485 | } else { |
486 | logit("nothing at all written"); | 486 | logit("nothing at all written"); |