From fb179bc2893b9f98844dc672390a53b219a79f26 Mon Sep 17 00:00:00 2001
From: Andrew Kaster <akaster@serenityos.org>
Date: Sun, 20 Feb 2022 01:42:54 -0700
Subject: [PATCH] Fuzzers: Avoid unnecessary ByteBuffer copies in FuzzWAVLoader

Avoid trying to memcpy from 0-byte sources as well, by bailing early on
nullptr data inputs.
---
 Meta/Lagom/Fuzzers/FuzzWAVLoader.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/Meta/Lagom/Fuzzers/FuzzWAVLoader.cpp b/Meta/Lagom/Fuzzers/FuzzWAVLoader.cpp
index 6e0c21f21f..92a183f4b2 100644
--- a/Meta/Lagom/Fuzzers/FuzzWAVLoader.cpp
+++ b/Meta/Lagom/Fuzzers/FuzzWAVLoader.cpp
@@ -10,8 +10,10 @@
 
 extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size)
 {
-    auto wav_data = ByteBuffer::copy(data, size).release_value();
-    auto wav = make<Audio::WavLoaderPlugin>(wav_data.bytes());
+    if (!data)
+        return 0;
+    auto wav_data = ReadonlyBytes { data, size };
+    auto wav = make<Audio::WavLoaderPlugin>(wav_data);
 
     for (;;) {
         auto samples = wav->get_more_samples();