AK: Unref old m_data in String's move assignment

We were overridding the data pointer without unreffing it,
causing a memory leak when assigning a String.

1 files changed, 3 insertions, 0 deletions

diff --git a/AK/String.cpp b/AK/String.cpp
index ac443a0ce5..c92c3c54eb 100644
--- a/AK/String.cpp
+++ b/AK/String.cpp
@@ -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;
 }