Closed Bug 240522 Opened 20 years ago Closed 20 years ago

ABR in PL_strlen, nsMsgSearchTerm::MatchArbitraryHeader(nsIMsgSearchScopeTerm *,UINT,UINT,char const*,int,nsIMsgDBHdr *,nsIMsgDatabase *,char const*,UINT,int,int *) [nsMsgSearchTerm.cpp:725]

Categories

(MailNews Core :: Filters, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

(Keywords: fixed1.7)

Attachments

(1 file)

If you get a message header > 512 bytes long, and you have a filter that works
on a non-default message header, we call PL_strlen on a non-null terminated
string. Fix upcoming.
Attached patch proposed fixSplinter Review
this showed up under purify - I think it's a recent regression from some code
to deal with multi-line headers.
Attachment #146115 - Flags: superreview?(mscott)
should block 1.7
Flags: blocking1.7?
Attachment #146115 - Flags: superreview?(mscott) → superreview+
Attachment #146115 - Flags: approval1.7?
fixed on trunk.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment on attachment 146115 [details] [diff] [review]
proposed fix

a=asa (on behalf of drivers) for checkin to 1.7
Attachment #146115 - Flags: approval1.7? → approval1.7+
Keywords: fixed1.7
Flags: blocking1.7?
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: