system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-10-07 00:19:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

AK: Fix leak in Optional(Optional&&)

Browse source 
We were *copying* the other Optional's value and then marking it as not
having a value. This prevented us from ever destroying the original.
This commit is contained in:
Andreas Kling 2019-08-05 22:27:47 +02:00
parent 9553ecfe01
commit 151e6a1818
2 changed files with 14 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
AK/Optional.h
View file

@ -29,7 +29,7 @@ public:
: m_has_value(other.m_has_value)
{
if (m_has_value) {
new (&m_storage) T(move(other.value_without_consume_state()));
new (&m_storage) T(other.release_value());
other.m_has_value = false;
}
}

13
AK/Tests/TestOptional.cpp
View file

@ -27,4 +27,17 @@ TEST_CASE(move_optional)
EXPECT_EQ(x.has_value(), false);
}
TEST_CASE(optional_leak_1)
{
struct Structure {
Optional<String> str;
};
// This used to leak, it does not anymore.
Vector<Structure> vec;
vec.append({ "foo" });
EXPECT_EQ(vec[0].str.has_value(), true);
EXPECT_EQ(vec[0].str.value(), "foo");
}
TEST_MAIN(Optional)