AK: Don't call memcpy with null argument in ByteBuffer::copy()

This was happening in TestBase64.test_decode, while copying an empty string.
Author: https://github.com/ADKaster Commit: https://github.com/SerenityOS/serenity/commit/11214bc94d3 Pull-request: https://github.com/SerenityOS/serenity/pull/7157 Issue: https://github.com/SerenityOS/serenity/issues/7073 Reviewed-by: https://github.com/linusg Reviewed-by: https://github.com/metmo
2025-04-21 20:15:17 +00:00 · 2021-05-16 13:46:33 -06:00 · 2021-05-16 13:46:33 -06:00 · 11214bc94d · 2024-07-18 17:59:56 +09:00
commit 11214bc94d
parent 00fa5dc53e
1 changed files with 2 additions and 1 deletions
--- a/AK/ByteBuffer.h
+++ b/AK/ByteBuffer.h
@ -74,7 +74,8 @@ public:
    [[nodiscard]] static ByteBuffer copy(void const* data, size_t size)
    {
        auto buffer = create_uninitialized(size);
-        __builtin_memcpy(buffer.data(), data, size);
+        if (size != 0)
+            __builtin_memcpy(buffer.data(), data, size);
        return buffer;
    }