From e2b9c61998a932638dd693e68bfa12e41f30114e Mon Sep 17 00:00:00 2001 From: David Vacca Date: Tue, 20 Apr 2021 12:11:25 -0700 Subject: [PATCH] Add assertion when building MapBuffer Summary: This diff adds an assertion when trying to build a MapBuffer with invalid data changelog: [internal] internal Reviewed By: sammy-SC Differential Revision: D27864309 fbshipit-source-id: 6601388e56be18ded0675f92cce009a577828c16 --- ReactCommon/react/renderer/mapbuffer/MapBufferBuilder.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/ReactCommon/react/renderer/mapbuffer/MapBufferBuilder.cpp b/ReactCommon/react/renderer/mapbuffer/MapBufferBuilder.cpp index ae2db84ff7..2aff6f0f48 100644 --- a/ReactCommon/react/renderer/mapbuffer/MapBufferBuilder.cpp +++ b/ReactCommon/react/renderer/mapbuffer/MapBufferBuilder.cpp @@ -160,6 +160,10 @@ void MapBufferBuilder::putMapBuffer(Key key, MapBuffer &map) { } MapBuffer MapBufferBuilder::build() { + react_native_assert( + (keyValues_ != nullptr) && + "Error when building mapbuffer with invalid datastructures."); + // Create buffer: [header] + [key, values] + [dynamic data] int bufferSize = keyValuesOffset_ + dynamicDataOffset_;