Using memcpy() in bstrcpy() confused FlexeLint, use strcpy() instead.

0fa4baea · Poul-Henning Kamp · e9a460f8 · 0fa4baea
Commit 0fa4baea authored Oct 21, 2019 by Poul-Henning Kamp
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

vdef.h include/vdef.h +2 -3

No files found.
--- a/include/vdef.h
+++ b/include/vdef.h
@@ -59,9 +59,8 @@
 /* Safe strcpy into a fixed-size buffer */
 #define bstrcpy(dst, src)						\
 	do {								\
-		size_t lbstrcpy = strlen(src) + 1;			\
+		assert(strlen(src) + 1 <= sizeof (dst));		\
-		assert(lbstrcpy <= sizeof dst);				\
+		strcpy((dst), (src));					\
-		memcpy(dst, src, lbstrcpy);				\
 	} while (0)
 // TODO #define strcpy BANNED