From 26a0ed5b0aab2b4f6cf6b786379b465bf09c9ffb Mon Sep 17 00:00:00 2001
From: giraffedata <giraffedata@9d0c8265-081b-0410-96cb-a4ca84ce46f8>
Date: Fri, 16 Nov 2012 16:23:09 +0000
Subject: Don't do overlapping strcpy()

git-svn-id: http://svn.code.sf.net/p/netpbm/code/trunk@1759 9d0c8265-081b-0410-96cb-a4ca84ce46f8
---
 lib/colorname.c | 23 +++++++++++++----------
 1 file changed, 13 insertions(+), 10 deletions(-)

(limited to 'lib/colorname.c')

diff --git a/lib/colorname.c b/lib/colorname.c
index 11df31c8..5c371e84 100644
--- a/lib/colorname.c
+++ b/lib/colorname.c
@@ -26,17 +26,20 @@
 
 static int lineNo;
 
+
+
 void 
-pm_canonstr(char * const str) {
-
-    char * p;
-    for (p = str; *p; ) {
-        if (ISSPACE(*p)) {
-            strcpy(p, &(p[1]));
-        } else {
-            if (ISUPPER(*p))
-                *p = tolower(*p);
-            ++p;
+pm_canonstr(char * const arg) {
+/*----------------------------------------------------------------------------
+   Modify string 'arg' to canonical form: lower case, no white space.
+-----------------------------------------------------------------------------*/
+    const char * srcCursor;
+    char * dstCursor;
+
+    for (srcCursor = arg, dstCursor = arg; *srcCursor; ++srcCursor) {
+        if (!ISSPACE(*srcCursor)) {
+            *dstCursor++ =
+                ISUPPER(*srcCursor) ? tolower(*srcCursor) : *srcCursor;
         }
     }
 }
-- 
cgit 1.4.1