693452 - crash [@ nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression]

Status: RESOLVED WORKSFORME

(MailNews Core :: Filters, defect)

Description

[Wayne Mery (:wsmwk)]

Mac+linux [@ nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression]
windows [@ nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression()]

code last touched several years ago 
bug 313357 - SeaMonkey crashes on quit (MailNews-related) [@ nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression()]
bug 349576 - Crash on editing filters [@ nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression]

bp-d3d636b7-004b-4aa7-9003-a08ac2111005 version 9 with reporter address
SIGSEGV
0x0
0       libxul.so       nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression mailnews/base/search/src/nsMsgLocalSearch.cpp:131  
1	libxul.so	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression	mailnews/base/search/src/nsMsgLocalSearch.cpp:131
2	libxul.so	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression	mailnews/base/search/src/nsMsgLocalSearch.cpp:131
3	libxul.so	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression	mailnews/base/search/src/nsMsgLocalSearch.cpp:131
4	libxul.so	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression	mailnews/base/search/src/nsMsgLocalSearch.cpp:131
5	libxul.so	nsMsgFilter::GetSearchTerms	mailnews/base/search/src/nsMsgFilter.cpp:459
6	libxul.so	nsImapMailFolder::UpdateFolderWithListener	mailnews/imap/src/nsImapMailFolder.cpp:736

Comment 1

[John Hopper]

I have been unable to reproduce this behavior but will watch more closely for this particular issue on my box.

Comment 2

[Wayne Mery (:wsmwk)]

bp-a1ecdf65-69d5-4342-8e29-6ba7a2121106 TB16.0.1
source line 98
97 // we must recursively destroy all sub expressions before we destroy ourself.....We leave search terms alone!
98 delete m_leftChild; 

A similar crash hits the next source line, 99, bp-bb9ef2eb-a9fd-480e-aef1-235f32121108  arena_dalloc_small | arena_dalloc | je_free | nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression()

Comment 3

[Wayne Mery (:wsmwk)]

Same location as comment 2

nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression() 
bp-fe9793a2-6ff5-4053-8aaa-c3cab2140207
Frances wrote  I had just opened Thunderbird and was reading the titles of my latest emails when it crashed 

0	XUL	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression()	mailnews/base/search/src/nsMsgLocalSearch.cpp
1	XUL	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression()	mailnews/base/search/src/nsMsgLocalSearch.cpp
2	XUL	nsMsgSearchBoolExpression::~nsMsgSearchBoolExpression()	mailnews/base/search/src/nsMsgLocalSearch.cpp
3	XUL	nsMsgFilter::~nsMsgFilter()	mailnews/base/search/src/nsMsgFilter.cpp
4	XUL	nsMsgFilter::~nsMsgFilter()	mailnews/base/search/src/nsMsgFilter.cpp
5	libSystem.B.dylib	free	
6		@0x1f317940	
7	XUL	nsMsgFilterAfterTheFact::~nsMsgFilterAfterTheFact()	mailnews/base/search/src/nsMsgFilterService.cpp
8	XUL	nsMsgFilterList::~nsMsgFilterList()	objdir-tb/i386/mozilla/dist/include/nsTArray.h
9	XUL	nsMsgFilterAfterTheFact::~nsMsgFilterAfterTheFact()	mailnews/base/search/src/nsMsgFilterService.cpp
10	XUL	nsMsgFilterList::Release()	mailnews/base/search/src/nsMsgFilterList.cpp
11	XUL	nsMsgFilterAfterTheFact::~nsMsgFilterAfterTheFact()	objdir-tb/i386/mozilla/dist/include/nsCOMPtr.h
12	XUL	nsMsgFilterAfterTheFact::~nsMsgFilterAfterTheFact()	mailnews/base/search/src/nsMsgFilterService.cpp
13	XUL	nsMsgFilterAfterTheFact::Release()	mailnews/base/search/src/nsMsgFilterService.cpp
14	XUL	nsMsgMailNewsUrl::SetUrlState(bool, tag_nsresult)	objdir-tb/i386/mozilla/dist/include/nsCOMPtr.h
15	XUL	_ZThn4_N9nsImapUrl14QueryInterfaceERK4nsIDPPv	mailnews/imap/src/nsImapUrl.cpp
16	XUL	nsCOMPtr_base::assign_from_qi(nsQueryInterface, nsID const&)	objdir-tb/i386/mozilla/xpcom/build/nsCOMPtr.cpp

Comment 4

[Ludovic Hirlimann [:Usul]]

Removing myslef on all the bugs I'm cced on. Please NI me if you need something on MailNews Core bugs from me.

Comment 5

[Wayne Mery (:wsmwk)]

Signature exists for version 60, but not for 68. (not even a variation of the signature)