Posted by Morbus Iff on May 2, 2008 at 1:30pm
| Project: | Mailhandler |
| Version: | master |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I have an email in the form of:
[... more header stuff ...]
Status: RO
X-Status:
X-Keywords:
X-UID: 1
: INTERNET:email@example.com wrote:
> blah blah blah blah
blah blah blahThe first line of the body - starting with :, is improperly considered a Mailhandler command line. Due to lack of input checking, this would then attempt to set $node->'' to the string (since there's no command word here). The attached patch does a check to ensure that the command word exists before attempting to assign to $node. This patch DOES NOT attempt to add the erroneous line back to the body (primarily because I didn't investigate how). I'm not sure your feelings on this.
| Attachment | Size |
|---|---|
| null_command_word.patch | 757 bytes |
Comments
#1
#2
committed to D5. Won't apply to D6 ... Thanks.
#3
Same patch. Untested. (Note: there's no Version for 6.x-dev).
#4
committed without testing :)
project module still hasn't published the 6.x-dev release so no Version here just yet.
#5
Automatically closed -- issue fixed for two weeks with no activity.