Add null pointer check for CopyInto. (#4671)

memcpy to a nullptr is technically undefined behavior. Add a null check to possibly avoid crashing. b/388593093
youtube · Jan 9, 2025 · 8616801 · 8616801
1 parent 400bbe9
commit 8616801
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/starboard/android/shared/media_decoder.cc b/starboard/android/shared/media_decoder.cc
@@ -475,7 +475,9 @@ bool MediaDecoder::ProcessOneInputBuffer(
       ReportError(kSbPlayerErrorDecode, error_message);
       return false;
     }
-    byte_buffer.CopyInto(data, size);
+    if (data) {
+      byte_buffer.CopyInto(data, size);
+    }
   }
 
   jint status;