From b2c6d0d0f64d419590c7cd77aae4890866665cba Mon Sep 17 00:00:00 2001
From: Alan Coopersmith <alan.coopersmith@oracle.com>
Date: Tue, 12 Feb 2013 18:36:30 -0800
Subject: Bug 56091 - Unsafe use of strcpy() in makedepend

Use memmove for potentially overlapping copies.

Reported-by: Laurence Jupp <laurence@narya.org>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Reviewed-by: Matthieu Herrb <matthieu.herrb@laas.fr>
---
 parse.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'parse.c')

diff --git a/parse.c b/parse.c
index 2d7c95a..e5240c3 100644
--- a/parse.c
+++ b/parse.c
@@ -229,7 +229,7 @@ deftype (char *line, struct filepointer *filep,
 		/*
 		 * copy the definition back to the beginning of the line.
 		 */
-		strcpy (line, p);
+		memmove (line, p, strlen(p) + 1);
 		break;
 	case ELSE:
 	case ENDIF:
-- 
cgit v1.2.1