[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Nmh-workers] mhfixmsg on a pathological mail
From: |
Ralph Corderoy |
Subject: |
Re: [Nmh-workers] mhfixmsg on a pathological mail |
Date: |
Wed, 30 Aug 2017 14:31:06 +0100 |
Hi Ken,
> Where it actually ends up segfaulting is in InitText(). The original
> code from mh-6.8.5 says:
>
> if (!*ci -> ci_subtype) /* XXX: attmail bogosity! */
> ci -> ci_subtype = add ("plain", ci -> ci_subtype);
...
> I guess maybe the idea was you could do:
>
> #<text
>
> Without having to specify #<text/plain ?
Well, that's the parsing side; making the `/subtype' optional. But the
above comment suggests RFC-violating `Content-Type: text' would arrive
from ATTMail when they sent to internet!nongnu.org!nmh-workers.
> Sigh. I think maybe ripping that bypass for the missing subtype would
> be best. Other thoughts?
There's three cases of /ci_subtype.*ci_subtype/ in the source.
mhfixmsg.c's allows for it to be NULL. The other two could easily do so
too. That would seem the easiest fix without needing to figure out all
the ways it could be reached.
I don't think this is a 1.7 thing; it's been there years already and
1.8 will be RSN.
--
Cheers, Ralph.
https://plus.google.com/+RalphCorderoy
- Re: [Nmh-workers] mhfixmsg on a pathological mail, (continued)
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ralph Corderoy, 2017/08/29
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ken Hornstein, 2017/08/29
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ken Hornstein, 2017/08/29
- Re: [Nmh-workers] mhfixmsg on a pathological mail, valdis . kletnieks, 2017/08/29
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ralph Corderoy, 2017/08/29
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ken Hornstein, 2017/08/30
- Re: [Nmh-workers] mhfixmsg on a pathological mail,
Ralph Corderoy <=
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ken Hornstein, 2017/08/30
- Re: [Nmh-workers] mhfixmsg on a pathological mail, Ralph Corderoy, 2017/08/19
Re: [Nmh-workers] mhfixmsg on a pathological mail, David Levine, 2017/08/18