Closed
Bug 18036
Opened 25 years ago
Closed 25 years ago
[DOGFOOD] arbitrary header in filter rule causes infinite loop in nsMsgSearchTerm.cpp
Categories
(MailNews Core :: Backend, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M11
People
(Reporter: sspitzer, Assigned: sspitzer)
Details
(Whiteboard: [PDT+])
Attachments
(1 file)
(deleted),
text/plain
|
Details |
this prevent me from opening my Inbox, where that message lives.
I'll attach the message next.
I'm debugging now. come stop by to see the gory details.
Assignee | ||
Comment 1•25 years ago
|
||
Assignee | ||
Updated•25 years ago
|
Summary: [DOGFOOD] message causing infinite loop in nsMsgSearchTerm.cpp → [DOGFOOD] arbitrary header in filter rule causes infinite loop in nsMsgSearchTerm.cpp
Assignee | ||
Comment 2•25 years ago
|
||
updating summary to reflect the real problem.
here's the rule that caused the problem:
name="mail qa auto cvs update"
enabled="yes"
description=""
type="1"
action="Move to folder"
actionValue="//imap:Trash"
condition=" AND (\"Message-ID\",contains,worksforme.mcom.com)"
in 4.x, on Linux at least, Message-ID was not one of the default header types to
filter on, so I created it.
Assignee | ||
Comment 3•25 years ago
|
||
the following patch fixes the problem.
Index: src/nsMsgBodyHandler.cpp
===================================================================
RCS file: /cvsroot/mozilla/mailnews/base/search/src/nsMsgBodyHandler.cpp,v
retrieving revision 1.7
diff -r1.7 nsMsgBodyHandler.cpp
163a164,166
> else if (m_headersSize == 0) {
> buf[0] = '\0';
> }
172d174
< char * line = NULL;
(that second bit just removes a warning.)
Assignee | ||
Comment 4•25 years ago
|
||
stealing from david, as I'm about to check in the fix.
Assignee: bienvenu → sspitzer
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M11
Assignee | ||
Comment 5•25 years ago
|
||
marking m11. accepting.
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•25 years ago
|
||
fixed.
OK using:
1999-11-09-11m11 commercial build NT 4.0
1999-11-09-11m11 commercial build linux 6.0
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•