Use memcpy() instead of memmove() when buffers are known not to overlap

Most of these came from a mass bcopy() -> memmove() substitution
in 1993 with a commit comment of "ANSIfication".

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/Error.c b/src/Error.c
index ed5990a..93e3c9e 100644
--- a/src/Error.c
+++ b/src/Error.c
@@ -266,7 +266,7 @@ DefaultMsg(String name,
 
             if (i > 10)
                 i = 10;
-            (void) memmove((char *) par, (char *) params, i * sizeof(String));
+            (void) memcpy(par, params, i * sizeof(String));
             memset(&par[i], 0, (10 - i) * sizeof(String));
             (void) fprintf(stderr, "%s%s",
                            error ? XTERROR_PREFIX : XTWARNING_PREFIX,
@@ -305,7 +305,7 @@ program as a non-root user or by removing the suid bit on the executable.");
 
         if (i > 10)
             i = 10;
-        (void) memmove((char *) par, (char *) params, i * sizeof(String));
+        (void) memcpy(par, params, i * sizeof(String));
         memset(&par[i], 0, (10 - i) * sizeof(String));
         if (i != *num_params)
             XtWarning("Some arguments in following message were lost");