635218 - crash [@ nsImapProtocol::UpdateFolderQuotaData(nsCString&, unsigned int, unsigned int)]

Status: RESOLVED WORKSFORME

(MailNews Core :: Networking: IMAP, defect)

Description

[Wayne Mery (:wsmwk)]

crash [@ nsImapProtocol::UpdateFolderQuotaData(nsCString&, unsigned int, unsigned int)]

typically happens shortly after startup - not a suprise.

bp-8d25cb20-533f-4aed-81eb-93b432110207
EXCEPTION_ACCESS_VIOLATION_READ
0x0
0	thunderbird.exe	nsImapProtocol::UpdateFolderQuotaData	mailnews/imap/src/nsImapProtocol.cpp:8396
1	thunderbird.exe	nsImapServerResponseParser::quota_data	mailnews/imap/src/nsImapServerResponseParser.cpp:2981
2	thunderbird.exe	nsImapServerResponseParser::response_data	mailnews/imap/src/nsImapServerResponseParser.cpp:751
3	thunderbird.exe	nsImapServerResponseParser::ParseIMAPServerResponse	mailnews/imap/src/nsImapServerResponseParser.cpp:246
4	thunderbird.exe	nsImapProtocol::ParseIMAPandCheckForNewMail	mailnews/imap/src/nsImapProtocol.cpp:1896
5	thunderbird.exe	nsImapProtocol::GetQuotaDataIfSupported	mailnews/imap/src/nsImapProtocol.cpp:8429
6	thunderbird.exe	nsImapProtocol::ProcessMailboxUpdate	mailnews/imap/src/nsImapProtocol.cpp:3895
7	thunderbird.exe	nsImapProtocol::ProcessSelectedStateURL	mailnews/imap/src/nsImapProtocol.cpp:2859
8	thunderbird.exe	nsImapProtocol::ProcessCurrentURL	mailnews/imap/src/nsImapProtocol.cpp:1741
9	thunderbird.exe	nsImapProtocol::ImapThreadMainLoop	mailnews/imap/src/nsImapProtocol.cpp:1380 

low crash rate. both v3.1.x and v3.0.x.
no contactable reporters. no crash comments.

Comment 1

[Wayne Mery (:wsmwk)]

only a couple per month

current example bp-83aee008-6385-471f-8535-772d02140718
0	xul.dll	nsImapProtocol::UpdateFolderQuotaData(nsCString &,unsigned int,unsigned int)	mailnews/imap/src/nsImapProtocol.cpp
1	xul.dll	nsImapServerResponseParser::quota_data()	mailnews/imap/src/nsImapServerResponseParser.cpp
2	xul.dll	nsImapServerResponseParser::response_data()	mailnews/imap/src/nsImapServerResponseParser.cpp
3	xul.dll	nsImapServerResponseParser::ParseIMAPServerResponse(char const *,bool,char *)	mailnews/imap/src/nsImapServerResponseParser.cpp
4	xul.dll	nsImapProtocol::ParseIMAPandCheckForNewMail(char const *,bool)	mailnews/imap/src/nsImapProtocol.cpp
5	xul.dll	nsImapProtocol::GetQuotaDataIfSupported(char const *)	mailnews/imap/src/nsImapProtocol.cpp
6	xul.dll	nsImapProtocol::ProcessMailboxUpdate(bool)	mailnews/imap/src/nsImapProtocol.cpp
7	xul.dll	nsImapProtocol::SelectMailbox(char const *)	mailnews/imap/src/nsImapProtocol.cpp 


http://hg.mozilla.org/releases/comm-esr24/annotate/a908efbe7f74/mailnews/imap/src/nsImapServerResponseParser.cpp#l3027
ehsan@13324 3008 uint32_t used, max;
hg@0 3009 char *parengroup;
hg@0 3010
hg@0 3011 AdvanceToNextToken();
hg@0 3012 if (ContinueParse())
hg@0 3013 {
hg@0 3014 nsCString quotaroot;
hg@0 3015 quotaroot.Adopt(CreateAstring());
hg@0 3016
hg@0 3017 if(ContinueParse() && !fAtEndOfLine)
hg@0 3018 {
hg@0 3019 AdvanceToNextToken();
hg@0 3020 if(fNextToken)
hg@0 3021 {
hg@0 3022 if(!PL_strcasecmp(fNextToken, "(STORAGE"))
hg@0 3023 {
hg@0 3024 parengroup = CreateParenGroup();
hg@0 3025 if(parengroup && (PR_sscanf(parengroup, "(STORAGE %lu %lu)", &used, &max) == 2) )
hg@0 3026 {
hg@0 3027 fServerConnection.UpdateFolderQuotaData(quotaroot, used, max); 

http://hg.mozilla.org/releases/comm-esr24/annotate/a908efbe7f74/mailnews/imap/src/nsImapProtocol.cpp#l8426
ehsan@13324 8422 void nsImapProtocol::UpdateFolderQuotaData(nsCString& aQuotaRoot, uint32_t aUsed, uint32_t aMax)
hg@0 8423 {
hg@0 8424 NS_ASSERTION(m_imapMailFolderSink, "m_imapMailFolderSink is null!");
hg@0 8425
hg@0 8426 m_imapMailFolderSink->SetFolderQuotaData(aQuotaRoot, aUsed, aMax);

Comment 2

[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 3

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

I'm marking this bug as WORKSFORME as bug crashlog signature didn't appear from a long time (over half year).