VM

Comment 8 for bug 612111

Revision history for this message
Uday Reddy (reddyuday) wrote :

Ok, I looked at what is happening.

According to the RFC 2046, sec.5.1, para 2:

Each part of a multipart/X is supposed to have a bunch of MIME headers, followed by a blank line, and then the body of the MIME part.

For multipart/digest, the body is in turn a message, which should have message headers, followed by a blank line, and then the body of the message.

The NPR digest that you have attached is not satisfying these criteria. The blank line following the MIME headers is missing. It is ok for the MIME headers to be missing. VM applies the defaults then. But the blank line should not be missing.

VM code has been fine. It is unable to parse the NPR digest because it is invalid. I am afraid we have to revert all your fixes.

If we want to allow this kind of thing, then the proper thing to do would be to alter the MIME parsing part of VM to allow for such deviations. If you can find some clear criteria whereby VM can guess that the sender is sending bad MIME, then we should discuss them and change VM accordingly.