Added support to load objects serialized by SEAL v3.4+.

microsoft · Mar 17, 2022 · 8d31897 · 8d31897
1 parent fda6250
commit 8d31897
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/native/src/seal/ciphertext.cpp b/native/src/seal/ciphertext.cpp
@@ -275,7 +275,10 @@ namespace seal
             double scale = 0;
             stream.read(reinterpret_cast<char *>(&scale), sizeof(double));
             uint64_t correction_factor = 1;
-            stream.read(reinterpret_cast<char *>(&correction_factor), sizeof(uint64_t));
+            if (version.major == 4)
+            {
+                stream.read(reinterpret_cast<char *>(&correction_factor), sizeof(uint64_t));
+            }
 
             // Set values already at this point for the metadata validity check
             new_data.parms_id_ = parms_id;

diff --git a/native/src/seal/serialization.h b/native/src/seal/serialization.h
@@ -150,16 +150,16 @@ namespace seal
             }
 
             // Different major versions not supported
-            if (header.version_major != SEAL_VERSION_MAJOR)
+            if (header.version_major != SEAL_VERSION_MAJOR && header.version_major != 3)
             {
                 return false;
             }
 
-            //            // Support Microsoft SEAL 3.4 and above
-            //            if (header.version_major == 3 && header.version_minor >= 4)
-            //            {
-            //                return true;
-            //            }
+            // Support Microsoft SEAL 3.4 and above
+            if (header.version_major == 3 && header.version_minor >= 4)
+            {
+                return true;
+            }
 
             return false;
         }