Project

General

Profile

Actions
Copy link

Bug #5338

closed
JL VJ

Bug #5316: smtp: PreProcessCommands does not handle all the edge cases

smtp: PreProcessCommands does not handle all the edge cases (6.0.x backport)

Bug #5338: smtp: PreProcessCommands does not handle all the edge cases (6.0.x backport)

Added by Jeff Lucovsky almost 4 years ago. Updated over 3 years ago.

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

Description

Issue
An edge case where <CRLF> came in fragmented for a line was not handled. This resulted in <CR> being treated as regular data byte and not as a part of the line end sequence. This particularly led to issues when base64 encoded data was processed by MIME and it could not handle the <CR> char as it relied on only receiving correct data from SMTP parser making file sizes different and inconsistent.

Context
As per RFC 2821 (SMTP),
a. Bare `CR` and `LF` have a long history of problems w mail implementations and applications.
b. An ideal SMTP client MUST NOT transmit `CR` and `LF` chars except when they are intended as line terminators and then also as `<CRLF>` sequence ONLY.
c. We should tolerate trailing `SP` before the terminating `<CRLF>`

JL Updated by Jeff Lucovsky almost 4 years ago Actions
Copy link
 #1

  • Copied from Bug #5316: smtp: PreProcessCommands does not handle all the edge cases added

JL Updated by Jeff Lucovsky almost 4 years ago Actions
Copy link
 #2

  • Subject changed from smtp: PreProcessCommands does not handle all the edge cases to Backport 6.0.x: smtp: PreProcessCommands does not handle all the edge cases
  • Parent task set to #5316

JL Updated by Jeff Lucovsky almost 4 years ago Actions
Copy link
 #3

  • Subject changed from Backport 6.0.x: smtp: PreProcessCommands does not handle all the edge cases to smtp: PreProcessCommands does not handle all the edge cases (6.0.x backport)

VJ Updated by Victor Julien almost 4 years ago Actions
Copy link
 #4

  • Status changed from Assigned to In Progress
  • Assignee changed from Shivani Bhardwaj to Victor Julien

VJ Updated by Victor Julien almost 4 years ago Actions
Copy link
 #5

  • Status changed from In Progress to In Review

VJ Updated by Victor Julien almost 4 years ago Actions
Copy link
 #6

  • Status changed from In Review to Closed

VJ Updated by Victor Julien over 3 years ago Actions
Copy link
 #7

  • Private changed from Yes to No
Actions
Copy link

Also available in: PDF Atom