Klocwork fix: avoiding using memcpy

This patch reworks the code to avoid
using the memcpy function.

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

Author: tcahuzax

test/test-subsystem/TESTSubsystemBinary.cpp

@@ -34,6 +34,7 @@
 #include <stdlib.h>
 #include <assert.h>
 #include <algorithm>
+#include <AlwaysAssert.hpp>
 #include <Iterator.hpp>
 #include "TESTSubsystemBinary.h"
 
@@ -51,9 +52,10 @@ std::string CTESTSubsystemBinary::toString(const void *pvValue, size_t size) con
     std::ostringstream strStream;
     uint32_t uiValue = 0;
 
-    assert(size <= sizeof(uiValue));
+    ALWAYS_ASSERT(size <= sizeof(uiValue), "Wrong size '" << size << "' should be <= to '"
+                                                          << sizeof(uiValue) << "'");
 
-    memcpy(&uiValue, pvValue, size);
+    uiValue = *static_cast<const uint32_t *>(pvValue);
 
     strStream << "0x" << std::hex << uiValue;