From: Daniel Stenberg <daniel@haxx.se>
Date: Sun, 21 Dec 2014 22:21:16 +0000 (+0100)
Subject: smb: use memcpy() instead of strncpy()
X-Git-Tag: curl-7_40_0~115
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2c5f346d3a18b829f2c9ce8dbb98470e41017ca3;p=curl

smb: use memcpy() instead of strncpy()

... as it never copies the trailing zero anyway and always just the four
bytes so let's not mislead anyone into thinking it is actually treated
as a string.

Coverity CID: 1260214
---

diff --git a/lib/smb.c b/lib/smb.c
index 9648f6519..a8fcf4acc 100644
--- a/lib/smb.c
+++ b/lib/smb.c
@@ -345,7 +345,7 @@ static void smb_format_message(struct connectdata *conn, struct smb_header *h,
   memset(h, 0, sizeof(*h));
   h->nbt_length = htons((unsigned short) (sizeof(*h) - sizeof(unsigned int) +
                                           len));
-  strncpy((char *)h->magic, "\xffSMB", 4);
+  memcpy((char *)h->magic, "\xffSMB", 4);
   h->command = cmd;
   h->flags = SMB_FLAGS_CANONICAL_PATHNAMES | SMB_FLAGS_CASELESS_PATHNAMES;
   h->flags2 = smb_swap16(SMB_FLAGS2_IS_LONG_NAME | SMB_FLAGS2_KNOWS_LONG_NAME);