diff options
| author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2021-09-08 01:03:34 +0000 |
|---|---|---|
| committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2021-09-08 01:03:34 +0000 |
| commit | 5eaedb51b9e618a1c522674b7c14b90778e6f32d (patch) | |
| tree | 564ea1ff891a65247faf4f50de01c42a187e4ce4 | |
| parent | Merge pull request #7893 (diff) | |
| download | monero-5eaedb51b9e618a1c522674b7c14b90778e6f32d.tar.xz | |
unit_tests: fix use after free in serialization test
| -rw-r--r-- | tests/unit_tests/serialization.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tests/unit_tests/serialization.cpp b/tests/unit_tests/serialization.cpp index 535752665..f4c73d3d5 100644 --- a/tests/unit_tests/serialization.cpp +++ b/tests/unit_tests/serialization.cpp @@ -132,7 +132,8 @@ TEST(Serialization, BinaryArchiveInts) { ASSERT_EQ(8, oss.str().size()); ASSERT_EQ(string("\0\0\0\0\xff\0\0\0", 8), oss.str()); - binary_archive<false> iar{epee::strspan<std::uint8_t>(oss.str())}; + const std::string s = oss.str(); + binary_archive<false> iar{epee::strspan<std::uint8_t>(s)}; iar.serialize_int(x1); ASSERT_EQ(8, iar.getpos()); ASSERT_TRUE(iar.good()); @@ -150,7 +151,8 @@ TEST(Serialization, BinaryArchiveVarInts) { ASSERT_EQ(6, oss.str().size()); ASSERT_EQ(string("\x80\x80\x80\x80\xF0\x1F", 6), oss.str()); - binary_archive<false> iar{epee::strspan<std::uint8_t>(oss.str())}; + const std::string s = oss.str(); + binary_archive<false> iar{epee::strspan<std::uint8_t>(s)}; iar.serialize_varint(x1); ASSERT_TRUE(iar.good()); ASSERT_EQ(x, x1); |