apparently developers are still not aware that they should be use ast_copy_string instead of strncpy... fix up many more users, and fix some bugs in the process

git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@46200 f38db490-d61c-443f-a65b-d21fe96a405b

1 files changed, 1 insertions, 2 deletions

diff --git a/main/db.c b/main/db.c
index b9c9115c2..ed6fbc683 100644
--- a/main/db.c
+++ b/main/db.c
@@ -202,8 +202,7 @@ int ast_db_get(const char *family, const char *keys, char *value, int valuelen)
 		if (data.size) {
 			((char *)data.data)[data.size - 1] = '\0';
 			/* Make sure that we don't write too much to the dst pointer or we don't read too much from the source pointer */
-			strncpy(value, data.data, (valuelen > data.size) ? data.size : valuelen);
-			value[valuelen - 1] = '\0';
+			ast_copy_string(value, data.data, (valuelen > data.size) ? data.size : valuelen);
 		} else {
 			ast_log(LOG_NOTICE, "Strange, empty value for /%s/%s\n", family, keys);
 		}