From f9b94dcafa867c38dfaeb9b24f37b392d1ad80d8 Mon Sep 17 00:00:00 2001 From: Simon Rettberg Date: Wed, 14 Jan 2015 13:55:34 +0100 Subject: [SERVER] Crank up warning levels of gcc and fix them all (mostly sign compare and some unused params) Also allow newline at end of alt-servers file without complaining about it not being a valid alt server definition. --- src/server/protocol.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/server/protocol.h') diff --git a/src/server/protocol.h b/src/server/protocol.h index b0a1f43..ad286d1 100644 --- a/src/server/protocol.h +++ b/src/server/protocol.h @@ -59,7 +59,7 @@ static inline bool dnbd3_select_image(int sock, char *lower_name, uint16_t rid, iov[0].iov_len = sizeof(request); iov[1].iov_base = &serialized; iov[1].iov_len = len; - return writev( sock, iov, 2 ) == len + sizeof(request); + return writev( sock, iov, 2 ) == len + (ssize_t)sizeof(request); } static inline bool dnbd3_get_block(int sock, uint64_t offset, uint32_t size, uint64_t handle) @@ -95,9 +95,9 @@ static inline bool dnbd3_get_crc32(int sock, uint32_t *master, void *buffer, siz if ( reply.cmd != CMD_GET_CRC32 || reply.size > *bufferLen ) return false; *bufferLen = reply.size; if ( recv( sock, master, sizeof(uint32_t), MSG_WAITALL | MSG_NOSIGNAL ) != sizeof(uint32_t) ) return false; - int done = 0; + uint32_t done = 0; while ( done < reply.size ) { - const int ret = recv( sock, (char*)buffer + done, reply.size - done, 0 ); + const ssize_t ret = recv( sock, (char*)buffer + done, reply.size - done, 0 ); if ( ret <= 0 ) return false; done += ret; } -- cgit v1.2.3-55-g7522