166196 - Crash in msgimap [@ nsImapProtocol::HandleMessageDownLoadLine]

Status: VERIFIED FIXED

(MailNews Core :: Networking: IMAP, defect)

Description

[Ian Neal]

Using BuildID 2002083108 on WinXP connecting to NS Messenger 4.15
I was clicking from one folder to another quite quickly when mozilla crashed.


MS Report gave this as the signature
AppName: mozilla.exe	 AppVer: 1.1.0.0	 ModName: msgimap.dll
ModVer: 1.1.0.0	 Offset: 0000820f

TBID TB10245821K

Comment 1

[Ian Neal]

might be related to bug 162342

Comment 2

[Jan Varga [:janv]]

Attachment: talkback data

Comment 3

[Olav Vitters]

Bug seems to be fixed (not yet verified). Was fixed around the same time as your
build id.

*** This bug has been marked as a duplicate of 164798 ***

Comment 4

[David :Bienvenu]

if this is happening in the 08/31 build, then I didn't fix it. I'll reopen this
one, since it has the start of steps to reproduce the problem. I was not able to
reproduce it by clicking quickly through folders, however.

Comment 5

[David :Bienvenu]

Attachment: (remotely) possible fix

we should be resetting this index when we finish downloading headers. I thought
there might be some sort of boundary condition when we download 9 headers or
so, then download another one. I could not reproduce the problem based on this
theory, but I'd like to check this fix in anyway, because it's the only thing I
can find wrong at the moment and it can't hurt.

Comment 6

[David :Bienvenu]

taking

Comment 7

[(not reading, please use [e-mail] instead)]

Comment on attachment 97607 [details] [diff] [review]
(remotely) possible fix

r=sspitzer

Comment 8

[David :Bienvenu]

last patch checked in - will keep looking at talkback reports.

Comment 9

[Asa Dotzler [:asa]]

The last incident in talkback was on 09/03 so it looks like this is fixed. 
http://www.people.fas.harvard.edu/~dbaron/mozilla/talkback10

Comment 10

[David :Bienvenu]

talkback shows this crash:

 2002090408   MozillaTrunk   Linux 2.4.18   2002-09-08 09:33:15  
nsImapProtocol::HandleMessageDownLoadLine() 2eb66188

so it's probably still around :-( it might be getting better

Comment 11

[greer]

It makes it to the Trunk topcrash lists. Marking it a topcrash so that we can
track it in the reports. Sorry I'm not seeing specific steps for this one, David.

Comment 12

[David :Bienvenu]

marking fixed. It's been 6 days since a daily build has shown this crash. So
unless we're just not getting talkback reports, I'm hopeful that this is fixed.

Comment 13

[yulian chang]

Used 20020912 trunk build on Win2K.
Quickly clicking through folders doesn't crash for me.

Marked Verified.