Fix for CHECK failure due to pointer description sometimes being larger than allocated string space

PiperOrigin-RevId: 501355568
This commit is contained in:
MediaPipe Team 2023-01-11 12:54:58 -08:00 committed by Copybara-Service
parent 54268594dd
commit 0e56bd38f3

View File

@ -87,7 +87,8 @@ void AddVectorSink(const std::string& stream_name, //
node->mutable_options()->MutableExtension( node->mutable_options()->MutableExtension(
CallbackPacketCalculatorOptions::ext); CallbackPacketCalculatorOptions::ext);
options->set_type(CallbackPacketCalculatorOptions::VECTOR_PACKET); options->set_type(CallbackPacketCalculatorOptions::VECTOR_PACKET);
char address[17]; // Up to 64-bit pointer in hex (16 characters) and an optional "0x" prepended.
char address[19];
int written = snprintf(address, sizeof(address), "%p", dumped_data); int written = snprintf(address, sizeof(address), "%p", dumped_data);
CHECK(written > 0 && written < sizeof(address)); CHECK(written > 0 && written < sizeof(address));
options->set_pointer(address); options->set_pointer(address);
@ -112,7 +113,8 @@ void AddPostStreamPacketSink(const std::string& stream_name,
node->mutable_options()->MutableExtension( node->mutable_options()->MutableExtension(
CallbackPacketCalculatorOptions::ext); CallbackPacketCalculatorOptions::ext);
options->set_type(CallbackPacketCalculatorOptions::POST_STREAM_PACKET); options->set_type(CallbackPacketCalculatorOptions::POST_STREAM_PACKET);
char address[17]; // Up to 64-bit pointer in hex (16 characters) and an optional "0x" prepended.
char address[19];
int written = snprintf(address, sizeof(address), "%p", post_stream_packet); int written = snprintf(address, sizeof(address), "%p", post_stream_packet);
CHECK(written > 0 && written < sizeof(address)); CHECK(written > 0 && written < sizeof(address));
options->set_pointer(address); options->set_pointer(address);