From 7947431ec1e837b2149e94b81e505fe4516ee5c6 Mon Sep 17 00:00:00 2001
From: dec05eba <dec05eba@protonmail.com>
Date: Mon, 15 Oct 2018 23:46:47 +0200
Subject: Fix missaligned read/write

---
 sibs/SafeDeserializer.hpp | 30 ++++--------------------------
 1 file changed, 4 insertions(+), 26 deletions(-)

(limited to 'sibs/SafeDeserializer.hpp')

diff --git a/sibs/SafeDeserializer.hpp b/sibs/SafeDeserializer.hpp
index 7eb8aaf..57d2eee 100644
--- a/sibs/SafeDeserializer.hpp
+++ b/sibs/SafeDeserializer.hpp
@@ -5,6 +5,7 @@
 #include "../utils.hpp"
 #include <cstring>
 #include <stdexcept>
+#include <algorithm>
 
 namespace sibs
 {
@@ -37,32 +38,9 @@ namespace sibs
             verifyExtractSize(typeSize);
             size -= typeSize;
             T result;
+            memmove(&result, data, typeSize);
             #if BYTE_ORDER == BIG_ENDIAN
-            switch(typeSize)
-            {
-                case 1:
-                    result = *(T*)data;
-                    break;
-                case 2:
-                    *(u16*)&result = byteswap(*(u16*)data);
-                    break;
-                case 4:
-                    *(u32*)&result = byteswap(*(u32*)data);
-                    break;
-                case 8:
-                    *(u64*)&result = byteswap(*(u64*)data);
-                    break;
-                default:
-                {
-                    for(int i = 0; i < typeSize; ++i)
-                    {
-                        ((char*)&result)[i] = data[typeSize - 1 - i];
-                    }
-                    break;
-                }
-            }
-            #else
-            result = *(T*)data;
+            std::reverse((char*)&result, (char*)&result + typeSize);
             #endif
             data += typeSize;
             return result;
@@ -77,7 +55,7 @@ namespace sibs
             {
                 verifyExtractSize(destinationSize);
                 size -= destinationSize;
-                memcpy(destination, data, destinationSize);
+                memmove(destination, data, destinationSize);
                 data += destinationSize;
             }
         }
-- 
cgit v1.2.3