From: Russell Bryant <russell@russellbryant.com>
Date: Mon, 8 Feb 2010 23:23:45 +0000 (+0000)
Subject: Use memmove() instead of memcpy() for a case where the buffers overlap.
X-Git-Tag: 11.0.0-beta1~3464
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a0d74cef6675b52f094a6475cd4d84b74d2feea3;p=thirdparty%2Fasterisk.git

Use memmove() instead of memcpy() for a case where the buffers overlap.

Once again, valgrind is freaking awesome.  That is all.


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@245610 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/main/pbx.c b/main/pbx.c
index 3b3b1cd6db..6a36f434d4 100644
--- a/main/pbx.c
+++ b/main/pbx.c
@@ -2927,7 +2927,7 @@ static const char *ast_str_substring(struct ast_str *value, int offset, int leng
 
 	if (offset > 0) {
 		/* Go ahead and chop off the beginning */
-		memcpy(ast_str_buffer(value), ast_str_buffer(value) + offset, ast_str_strlen(value) - offset + 1);
+		memmove(ast_str_buffer(value), ast_str_buffer(value) + offset, ast_str_strlen(value) - offset + 1);
 		lr -= offset;
 	}