bugThe nmh Mail Handling System - Bugs: bug #15245, mhshow could handle invalid...

 
 

bug #15245: mhshow could handle invalid quoted-printable more gracefully

Submitter:  Peter Maydell <pm215>
Submitted:  Mon 19 Dec 2005 03:54:56 PM UTC
   
 
Category:  None Severity:  2 - Minor
Priority:  * 5 - Normal Status:  Fixed
Assigned to:  epg Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 13 Aug 2008 01:03:07 AM UTC, comment #2: 

Fixed, will be in next release.

/sources/nmh/nmh/ChangeLog,v  <--  ChangeLog
new revision: 1.289; previous revision: 1.288
/sources/nmh/nmh/test/tests/mhshow/test-qp,v  <--  test/tests/mhshow/test-qp
initial revision: 1.1
/sources/nmh/nmh/uip/mhparse.c,v  <--  uip/mhparse.c
new revision: 1.19; previous revision: 1.18

cvsps PatchSet 667

Eric Gillespie <epg>
Group Member
Thu 07 Aug 2008 04:13:36 PM UTC, comment #1: 

I have a fix for this in my tree. I'll make sure to get it into nmh along with the others.

Eric Gillespie <epg>
Group Member
Mon 19 Dec 2005 03:54:56 PM UTC, original submission:  

Watch:

===begin===
mnementh$ cat 444.2.txt
From: -email is unavailable-
To: -email is unavailable-
Subject: test
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Date: Sun, 18 Dec 2005 00:52:39 +0100

=cl
mnementh$ mhshow -file 444.2.txt
Date:    Sun, 18 Dec 2005 00:52:39 +0100
To:      -email is unavailable-
From:    -email is unavailable-
Subject: test

MIME-Version: 1.0

mhshow: invalid QUOTED-PRINTABLE encoding -- expecting hexidecimal-digit,
        but got char 0x6c
        (content text/plain in message /tmp/444.2.txt)
===endit===

It's right that =cl is invalid quoted-printable, but bombing out without showing anything is not very friendly. RFC 2045 section 6.7 (http://www.faqs.org/rfcs/rfc2045.html) recommends treating this kind of sequence literally (possibly with a warning). We should do this, and follow the other recommendations in that section.

PS: 'hexadecimal' is spelt thusly.

Peter Maydell <pm215>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by epg (Posted a comment)
  •  

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-08-21 epg Open/ClosedOpen Closed
    2008-08-13 epg StatusIn Progress Fixed
    2008-08-07 epg StatusNone In Progress
        Assigned toNone epg

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code