[PATCH 1/4] maint: dfa: avoid unnecessary uses of strcpy/strncpy

[Jim Meyering]

FYI, no semantic change:

>From 6e6c1fb5e8c7fc7fe260e809e3bcdc0927d519cf Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Fri, 13 Apr 2012 09:46:24 +0200
Subject: [PATCH 1/4] maint: dfa: avoid unnecessary uses of strcpy/strncpy

* src/dfa.c (icatalloc): Use memcpy, not strcpy, given the length.
(dfamust): Combine MALLOC+strcpy into cleaner xmemdup.
(parse_bracket_exp): Likewise, but replace a use of strncpy.
---
 src/dfa.c |    9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/dfa.c b/src/dfa.c
index 613f548..cd243fb 100644
--- a/src/dfa.c
+++ b/src/dfa.c
@@ -1024,9 +1024,7 @@ parse_bracket_exp (void)

               else if (MBS_SUPPORT && (c1 == '=' || c1 == '.'))
                 {
-                  char *elem;
-                  MALLOC (elem, len + 1);
-                  strncpy (elem, str, len + 1);
+                  char *elem = xmemdup (str, len + 1);

                   if (c1 == '=')
                     /* build equivalent class.  */
@@ -3667,7 +3665,7 @@ icatalloc (char *old, char const *new)
   if (newsize == 0)
     return old;
   result = xrealloc (old, oldsize + newsize + 1);
-  strcpy (result + oldsize, new);
+  memcpy (result + oldsize, new, newsize + 1);
   return result;
 }

@@ -4060,8 +4058,7 @@ done:
     {
       MALLOC (dm, 1);
       dm->exact = exact;
-      MALLOC (dm->must, strlen (result) + 1);
-      strcpy (dm->must, result);
+      dm->must = xmemdup (result, strlen (result) + 1);
       dm->next = d->musts;
       d->musts = dm;
     }
--
1.7.10.130.g36e6c