From 90879643717a1f8fef5061345e5e9b279329619f Mon Sep 17 00:00:00 2001 From: "Andrii Kalinich (GitHub)" Date: Tue, 24 Aug 2021 13:23:38 -0400 Subject: Ignore frame data for single and first frames (#3762) Co-authored-by: VladSemenyuk --- src/components/protocol_handler/src/multiframe_builder.cc | 1 + src/components/protocol_handler/src/protocol_packet.cc | 13 ------------- 2 files changed, 1 insertion(+), 13 deletions(-) (limited to 'src') diff --git a/src/components/protocol_handler/src/multiframe_builder.cc b/src/components/protocol_handler/src/multiframe_builder.cc index 875b5b5933..b66ce9b911 100644 --- a/src/components/protocol_handler/src/multiframe_builder.cc +++ b/src/components/protocol_handler/src/multiframe_builder.cc @@ -197,6 +197,7 @@ RESULT_CODE MultiFrameBuilder::HandleFirstFrame(const ProtocolFramePtr packet) { << connection_id << ", session_id: " << static_cast(session_id) << ", message_id: " << message_id); + packet->set_frame_data(FRAME_DATA_FIRST); messageId_map[message_id] = {packet, date_time::getCurrentTime()}; return RESULT_OK; } diff --git a/src/components/protocol_handler/src/protocol_packet.cc b/src/components/protocol_handler/src/protocol_packet.cc index 9c3dd03141..51b5e838b3 100644 --- a/src/components/protocol_handler/src/protocol_packet.cc +++ b/src/components/protocol_handler/src/protocol_packet.cc @@ -325,21 +325,8 @@ RESULT_CODE ProtocolPacket::ProtocolHeaderValidator::validate( break; } case FRAME_TYPE_SINGLE: - if (header.frameData != FRAME_DATA_SINGLE) { - SDL_LOG_WARN("FRAME_TYPE_SINGLE - Invalide frame data " - << static_cast(header.frameData)); - return RESULT_FAIL; - } - break; case FRAME_TYPE_FIRST: - if (header.frameData != FRAME_DATA_FIRST) { - SDL_LOG_WARN("FRAME_TYPE_FIRST - Invalide frame data " - << static_cast(header.frameData)); - return RESULT_FAIL; - } - break; case FRAME_TYPE_CONSECUTIVE: - // Could have any FrameInfo value break; default: SDL_LOG_WARN("Unknown frame type " << static_cast(header.frameType)); -- cgit v1.2.1