Klocwork fix: removing strncpy call

This patch reworks the code to avoid calling
the strncpy method, which is considered as
unsafe on windows.

Signed-off-by: Thomas Cahuzac <[email protected]>

Author: tcahuzax

test/test-subsystem/TESTSubsystemString.cpp

@@ -28,6 +28,8 @@
 * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 */
 #include <string.h>
+#include <iterator>
+#include <Iterator.hpp>
 #include "TESTSubsystemString.h"
 
 #define base CTESTSubsystemObject
@@ -46,5 +48,6 @@ std::string CTESTSubsystemString::toString(const void *pvValue, size_t /*size*/)
 
 void CTESTSubsystemString::fromString(const std::string &strValue, void *pvValue, size_t size)
 {
-    strncpy((char *)pvValue, strValue.c_str(), size);
+    auto destination = MAKE_ARRAY_ITERATOR(static_cast<char *>(pvValue), size);
+    std::copy_n(std::begin(strValue), size, destination);
 }