don't violate strict aliasing with reinterpret_cast

Signed-off-by: Tom Peters <[email protected]>

Author: tdp2110

Diff for: plugin/common/plugin.h

@@ -19,6 +19,7 @@
 #include "checkMacrosPlugin.h"
 
 #include "NvInferPlugin.h"
+#include <cstring>
 #include <cuda_runtime.h>
 #include <iostream>
 #include <memory>
@@ -78,15 +79,16 @@ class BaseCreator : public IPluginCreator
 template <typename T>
 void write(char*& buffer, const T& val)
 {
-    *reinterpret_cast<T*>(buffer) = val;
+    std::memcpy(buffer, &val, sizeof(T));
     buffer += sizeof(T);
 }
 
 // Read values from buffer
 template <typename T>
 T read(const char*& buffer)
 {
-    T val = *reinterpret_cast<const T*>(buffer);
+    T val;
+    std::memcpy(&val, buffer, sizeof(T));
     buffer += sizeof(T);
     return val;
 }

Diff for: samples/opensource/sampleUffPluginV2Ext/sampleUffPluginV2Ext.cpp

@@ -18,10 +18,10 @@
 #include "NvUffParser.h"
 #include <cassert>
 #include <chrono>
+#include <cstring>
 #include <cudnn.h>
 #include <iostream>
 #include <map>
-#include <string.h>
 #include <unordered_map>
 #include <vector>
 
@@ -579,14 +579,15 @@ class UffPoolPluginV2 : public IPluginV2IOExt
     template <typename T>
     void write(char*& buffer, const T& val) const
     {
-        *reinterpret_cast<T*>(buffer) = val;
+        std::memcpy(buffer, &val, sizeof(T));
         buffer += sizeof(T);
     }
 
     template <typename T>
     T read(const char*& buffer) const
     {
-        T val = *reinterpret_cast<const T*>(buffer);
+        T val;
+        std::memcpy(&val, buffer, sizeof(T));
         buffer += sizeof(T);
         return val;
     }