bugThe nmh Mail Handling System - Bugs: bug #1466, burst uses incorrect message...

 
 

bug #1466: burst uses incorrect message separator

Submitter:  Scott Lipcon <slipcon>
Submitted:  Mon 21 Oct 2002 02:55:37 PM UTC
   
 
Category:  Bug Severity:  3 - Normal
Priority:  * 5 - Normal Status:  Fixed
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 28 Feb 2012 03:05:24 AM UTC, comment #3: 

Fixed!  Just shy of a decade late.

According to RFC 934, a valid message boundary is any line starting with a '-' but not followed by a space.  I changed the code so it should accept anything valid, not just the mailman delimiter.

Ken Hornstein <kenh>
Group administrator
Wed 14 Jan 2004 10:36:07 AM UTC, comment #2: 

My email address is mark + nmh @ lanfear . net. Savannah fubar'ed it. Maybe the next cracker will fix Savannah's email parsing routines.

Anonymous
Wed 14 Jan 2004 10:33:58 AM UTC, comment #1: 

burst.patch hacks on support for mailman's message delimiter. Worksforme. Sorry not gonna create a Svannah account. <mark+nmh@lanfear.net>

Anonymous
Mon 21 Oct 2002 02:55:37 PM UTC, original submission:  

From comp.mail.mh via Anders Eriksson.  This bug is in nmh too.

"Bill Wohler" <wohler@newt.com> wrote in message news:<808z14tdw9.fsf@
gbr.newt.com>...

> Jym Dyer <jym@econet.org> writes:
> > > mailman is producing "--__--__--" as a message delimiter, and
> > > burst is looking for "-------".  i didn't try to mentally run
> > > the rfc934 state machine on that to see if it's legit or not.
> >
> > =v= Well swat my hind, the string is hardcoded into burst.c:
> >
> > static char delim3[] = "-------";
> >
> > Whereas RFC934 lets anything that starts with "-" but not with
> > "- " serve as a separator.
>
>   Thanks for digging into that. Turns out that MH has the bug, not
>   mailman.
>
> --
> Bill Wohler <wohler@newt.com>  http://www.newt.com/wohler/  GnuPG ID:610BD9AD
> Maintainer of comp.mail.mh FAQ and mh-e. Vote Libertarian!
> If you're passed on the right, you're in the wrong lane.



Scott Lipcon <slipcon>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #974:  burst.patch added by None (2KiB - application/octet-stream - Patch to add mailman)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by kenh (Posted a comment)
  • -email is unavailable- added by pm215 (Updated the item)
  •  

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2012-02-28 kenh StatusNone Fixed
        Open/ClosedOpen Closed
    2008-12-29 pm215 Categorycompatibility Bug
    2004-01-14 None Attached File- Added burst.patch, #954

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code