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

 
 

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

Submitted by:  Peter Maydell <pm215>
Submitted on:  Mon 19 Dec 2005 03:54:56 PM UTC  
 
Category: NoneSeverity: 2 - Minor
Priority: 5 - NormalStatus: Fixed
Assigned to: Eric Gillespie <epg>Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project MemberIn charge of this item.
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>
Project MemberIn charge of this item.
Mon 19 Dec 2005 03:54:56 PM UTC, original submission:

Watch:

===begin===
mnementh$ cat 444.2.txt
From: -unavailable-
To: -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: -unavailable-
From: -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>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 21 Aug 2008 12:02:03 AM UTCepgOpen/ClosedOpen=>Closed
    Wed 13 Aug 2008 01:03:07 AM UTCepgStatusIn Progress=>Fixed
    Thu 07 Aug 2008 09:44:39 PM UTCepgStatusNone=>In Progress
      Assigned toNone=>epg

    Back to the top


    Powered by Savane 3.1-cleanup1