🚑 fix for #516 and #518

We should compare the binary serializations rather than the JSON values themselves. This fix was already done for CBOR and apparently forgotten for MessagePack.

🚑 fix for #516 and #518
f547679d · Niels Lohmann · 2d474b52 · f547679d
Unverified Commit f547679d authored Mar 17, 2017 by Niels Lohmann
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

fuzzer-parse_msgpack.cpp test/src/fuzzer-parse_msgpack.cpp +2 -2

No files found.
--- a/test/src/fuzzer-parse_msgpack.cpp
+++ b/test/src/fuzzer-parse_msgpack.cpp
@@ -41,8 +41,8 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size)
            // parse serialization
            json j2 = json::from_msgpack(vec2);

-            // deserializations must match
-            assert(j1 == j2);
+            // serializations must match
+            assert(json::to_msgpack(j2) == vec2);
        }
        catch (const json::parse_error&)
        {