Closed Bug 525052 Opened 13 years ago Closed 13 years ago

crash running filter [@ nsMsgFilterService::ApplyFilters(int, nsIArray*, nsIMsgFolder*, nsIMsgWindow*)]

Categories

(MailNews Core :: Filters, defect)

1.9.1 Branch
x86
All
defect
Not set
critical

Tracking

(Not tracked)

VERIFIED FIXED
Thunderbird 3.0rc1

People

(Reporter: wsmwk, Assigned: rkent)

Details

(Keywords: crash, fixed-seamonkey2.0.1, regression)

Crash Data

Attachments

(1 file)

crash running filter [@ nsMsgFilterService::ApplyFilters(int, nsIArray*, nsIMsgFolder*, nsIMsgWindow*)]

3.0b4. #73 in rank. So not a strong candidate for blocking.
Doesn't appear in 3.0b3, so probably regression.
But no crashes in nightlies, so not possible to say if it's fixed on trunk
xref recently fixed  Bug 518565 [@nsMsgFilterList::LoadTextFilters(nsIInputStream*) ] (which isn't fixed in 3.0b4)

bp-8b22f32b-fc9e-4ea3-97bf-79ec12090930
0	thunderbird.exe	nsMsgFilterService::ApplyFilters	 mailnews/base/search/src/nsMsgFilterService.cpp:1012
1	xpcom_core.dll	NS_InvokeByIndex_P	xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:101
2	thunderbird.exe	XPCWrappedNative::CallMethod	js/src/xpconnect/src/xpcwrappednative.cpp:2295
3	thunderbird.exe	XPC_WN_CallMethod	js/src/xpconnect/src/xpcwrappednativejsops.cpp:1590 

All available comments:
- Was trying to move messages to newly created folder which used to be easy, did a search for emails containing whatever you used to create the folder, it wold then display them all, do a select all and then you could move them enamase to the new folder
- was running a filter on the message
- I searched messages then selected them and tried to run a message filter on them
- Did a search... click show all as list... then selected all messages and clicked run filter.
There is probably some UI issue that is calling filters in an invalid situation, but for now we should at least be able to stop the crash by doing the standard practice of checking for null pointers.
Assignee: nobody → kent
Status: NEW → ASSIGNED
Attachment #408941 - Flags: superreview?(bienvenu)
Attachment #408941 - Flags: review?(bienvenu)
Comment on attachment 408941 [details] [diff] [review]
Rev a: ENSURE_ARG_POINTER on folder

thx, Kent
Attachment #408941 - Flags: superreview?(bienvenu)
Attachment #408941 - Flags: superreview+
Attachment #408941 - Flags: review?(bienvenu)
Attachment #408941 - Flags: review+
Comment on attachment 408941 [details] [diff] [review]
Rev a: ENSURE_ARG_POINTER on folder

This is pretty low risk for RC1.
Attachment #408941 - Flags: approval-thunderbird3?
Attachment #408941 - Flags: approval-thunderbird3? → approval-thunderbird3+
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0rc1
crash is not seen in v3.0.0 (but is in 3.0b4) so I would say this crash is indeed gone
Status: RESOLVED → VERIFIED
Crash Signature: [@ nsMsgFilterService::ApplyFilters(int, nsIArray*, nsIMsgFolder*, nsIMsgWindow*)]
You need to log in before you can comment on or make changes to this bug.