Merge pull request #7923

5eaedb5 unit_tests: fix use after free in serialization test (moneromooo-monero)
This commit is contained in:
luigi1111 2021-09-23 22:16:38 -05:00
commit e7a1a095ac
No known key found for this signature in database
GPG Key ID: F4ACA0183641E010

View File

@ -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);