Project

General

Profile

Actions

Bug #6592

closed

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

Added by Philippe Antoine about 1 year ago. Updated 10 months 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
Actions #1

Updated by OISF Ticketbot about 1 year ago

  • Subtask #6593 added
Actions #2

Updated by OISF Ticketbot about 1 year ago

  • Label deleted (Needs backport to 7.0)
Actions #3

Updated by Jeff Lucovsky 11 months ago

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

Updated by Jeff Lucovsky 11 months ago

  • Status changed from In Progress to In Review
Actions #6

Updated by Philippe Antoine 10 months ago

  • Status changed from In Review to Resolved
Actions #7

Updated by Philippe Antoine 10 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF