Closed (fixed)
Project:
Mailhandler
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 May 2008 at 13:30 UTC
Updated:
16 May 2008 at 15:32 UTC
Jump to comment: Most recent file
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.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | null_command_word_d6.patch | 780 bytes | morbus iff |
| null_command_word.patch | 757 bytes | morbus iff |
Comments
Comment #1
morbus iffComment #2
moshe weitzman commentedcommitted to D5. Won't apply to D6 ... Thanks.
Comment #3
morbus iffSame patch. Untested. (Note: there's no Version for 6.x-dev).
Comment #4
moshe weitzman commentedcommitted without testing :)
project module still hasn't published the 6.x-dev release so no Version here just yet.
Comment #5
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.