]> granicus.if.org Git - curl/commitdiff
smb.c: Fixed code analysis warning
authorSteve Holme <steve_holme@hotmail.com>
Sun, 14 Dec 2014 21:01:09 +0000 (21:01 +0000)
committerSteve Holme <steve_holme@hotmail.com>
Sun, 14 Dec 2014 21:01:09 +0000 (21:01 +0000)
smb.c:320: warning C6297: Arithmetic overflow: 32-bit value is shifted,
           then cast to 64-bit value. Result may not be an expected
           value

lib/smb.c

index b310b0e1763fb9a97032993b1f6c49da33443667..9648f6519f3b8603bd391ed48c25f122a131fcdc 100644 (file)
--- a/lib/smb.c
+++ b/lib/smb.c
@@ -317,7 +317,7 @@ static CURLcode smb_recv_message(struct connectdata *conn, void **msg)
     if(nbt_size >= msg_size + sizeof(unsigned short)) {
       /* Add the byte count */
       msg_size += sizeof(unsigned short) + ((unsigned char) buf[msg_size]) +
-                  (((unsigned char) buf[msg_size + 1]) << 8);
+                         (((size_t) ((unsigned char) buf[msg_size + 1])) << 8);
       if(nbt_size < msg_size)
         return CURLE_READ_ERROR;
     }