Project

General

Profile

Actions

Bug #6592

closed
PA JL

mqtt: frames on TCP are not set properly when parsing multiple PDUs in one go

Bug #6592: mqtt: frames on TCP are not set properly when parsing multiple PDUs in one go

Added by Philippe Antoine over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

        while !current.is_empty() {
            SCLogDebug!("request: handling {}", current.len());
            match parse_message(current, self.protocol_version, self.max_msg_len) {
                Ok((rem, msg)) => {
                    let _pdu = Frame::new(
                        flow,
                        &stream_slice,
                        input,
                        current.len() as i64,
                        MQTTFrameType::Pdu as u8,
                    );

The code uses input which is the whole TCP stream, instead of current which is the current PDU


Subtasks 1 (0 open1 closed)

Bug #6593: mqtt: frames on TCP are not set properly when parsing multiple PDUs in one go (7.0.x backport)ClosedJeff LucovskyActions

OT Updated by OISF Ticketbot over 2 years ago Actions #1

  • Subtask #6593 added

OT Updated by OISF Ticketbot over 2 years ago Actions #2

  • Label deleted (Needs backport to 7.0)

JL Updated by Jeff Lucovsky about 2 years ago Actions #3

  • Status changed from New to In Progress
  • Assignee changed from OISF Dev to Jeff Lucovsky

JL Updated by Jeff Lucovsky about 2 years ago Actions #5

  • Status changed from In Progress to In Review

PA Updated by Philippe Antoine about 2 years ago Actions #6

  • Status changed from In Review to Resolved

PA Updated by Philippe Antoine about 2 years ago Actions #7

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom