kasan: No KASAN's memmove check if archs don't have it.

Nick Hu · palmer-dabbelt · commit 57ee58e39321 · 2020-01-22T13:09:41.000-08:00
If archs don't have memmove then the C implementation from lib/string.c is used,
and then it's instrumented by compiler. So there is no need to add KASAN's
memmove to manual checks.

Signed-off-by: Nick Hu &lt;nickhu@andestech.com&gt;
Acked-by: Dmitry Vyukov &lt;dvyukov@google.com&gt;
Acked-by: Andrey Ryabinin &lt;aryabinin@virtuozzo.com&gt;
Signed-off-by: Palmer Dabbelt &lt;palmerdabbelt@google.com&gt;
diff --git a/mm/kasan/common.c b/mm/kasan/common.c
@@ -110,6 +110,7 @@ void *memset(void *addr, int c, size_t len)
 	return __memset(addr, c, len);
 }
 
+#ifdef __HAVE_ARCH_MEMMOVE
 #undef memmove
 void *memmove(void *dest, const void *src, size_t len)
 {
@@ -118,6 +119,7 @@ void *memmove(void *dest, const void *src, size_t len)
 
 	return __memmove(dest, src, len);
 }
+#endif
 
 #undef memcpy
 void *memcpy(void *dest, const void *src, size_t len)

Original file line number	Diff line number	Diff line change
`@@ -110,6 +110,7 @@ void memset(void addr, int c, size_t len)`
`110`	`110`	`return __memset(addr, c, len);`
`111`	`111`	`}`
`112`	`112`
	`113`	`+#ifdef __HAVE_ARCH_MEMMOVE`
`113`	`114`	`#undef memmove`
`114`	`115`	`void memmove(void dest, const void *src, size_t len)`
`115`	`116`	`{`
`@@ -118,6 +119,7 @@ void memmove(void dest, const void *src, size_t len)`
`118`	`119`
`119`	`120`	`return __memmove(dest, src, len);`
`120`	`121`	`}`
	`122`	`+#endif`
`121`	`123`
`122`	`124`	`#undef memcpy`
`123`	`125`	`void memcpy(void dest, const void *src, size_t len)`