Merge pull request #134 from svigerske/fix-misaligned-load

avoid load of misaligned address warning in BinaryReader
ampl · Mar 22, 2021 · 8bfe66e · 8bfe66e
2 parents bb7d616 + 791e536
commit 8bfe66e
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/include/mp/nl-reader.h b/include/mp/nl-reader.h
@@ -1243,7 +1243,9 @@ class BinaryReader : private InputConverter, public BinaryReaderBase {
   template <typename Int>
   Int ReadInt() {
     token_ = ptr_;
-    return this->Convert(*reinterpret_cast<const Int*>(Read(sizeof(Int))));
+    Int val;
+    memcpy(&val, Read(sizeof(Int)), sizeof(Int));
+    return this->Convert(val);
   }
 
   int ReadUInt() {
@@ -1255,8 +1257,9 @@ class BinaryReader : private InputConverter, public BinaryReaderBase {
 
   double ReadDouble() {
     token_ = ptr_;
-    return this->Convert(
-          *reinterpret_cast<const double*>(Read(sizeof(double))));
+    double val;
+    memcpy(&val, Read(sizeof(double)), sizeof(double));
+    return this->Convert(val);
   }
 
   fmt::StringRef ReadString() {