system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-25 03:56:08 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

AK: Unref old m_data in String's move assignment

Browse source 
We were overridding the data pointer without unreffing it,
causing a memory leak when assigning a String.
This commit is contained in:
Maciej 2022-12-08 18:30:04 +01:00 committed by Andreas Kling
parent 741138c585
commit 58f5deba70
2 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
AK/String.cpp
View file

@ -181,6 +181,9 @@ String::String(String&& other)
String& String::operator=(String&& other)
{
if (!is_short_string())
m_data->unref();
m_data = exchange(other.m_data, nullptr);
return *this;
}

7
Tests/AK/TestString.cpp
View file

@ -24,6 +24,13 @@ TEST_CASE(construct_empty)
EXPECT_EQ(empty, ""sv);
}
TEST_CASE(move_assignment)
{
String string1 = MUST(String::from_utf8("hello"sv));
string1 = MUST(String::from_utf8("friends!"sv));
EXPECT_EQ(string1, "friends!"sv);
}
TEST_CASE(short_strings)
{
#ifdef AK_ARCH_64_BIT