#136 new mail_fetch class detects message end incorrectly

closed-fixed
nobody
None
5
2007-05-30
2006-12-23
No

command_retr() method in mail_fetch class uses (trim($line)=='.') to detect end of message. trim command can strip spaces before period.

Method should check for ".\r\n" string.

Discussion

  • Tomas Kuliavas

    Tomas Kuliavas - 2006-12-23

    Logged In: YES
    user_id=225877
    Originator: YES

    Related issue present in both mail_fetch (devel 1.5) and POP3 (stable 1.4) classes. If message line starts with . and is not equal to ".\r\n", pop3 server prepends one more period to that line and first period must be ignored. RFC 1939 3. Basic Operation chapter. 'When examining a multi-line response, the client checks to see if the line begins with the termination octet. If so and if octets other than CRLF follow, the first octet of the line (the termination octet) is stripped away.'

     
  • Thijs Kinkhorst

    Thijs Kinkhorst - 2007-05-30

    Logged In: YES
    user_id=285765
    Originator: NO

    Fix committed, thanks for reporting!

     
  • Thijs Kinkhorst

    Thijs Kinkhorst - 2007-05-30
    • status: open --> closed-fixed
     

Log in to post a comment.