Closed
Bug 134961
Opened 22 years ago
Closed 22 years ago
Crash in nsMsgNewsFolder::RemoveMessage
Categories
(MailNews Core :: Networking: NNTP, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.0
People
(Reporter: mozilla-bugs, Assigned: Bienvenu)
Details
(Keywords: crash, Whiteboard: [adt3] have patch)
Attachments
(1 file)
654 bytes,
patch
|
naving
:
review+
Bienvenu
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
I failed to reproduce it so far, but it happened when I clicked on the list-ids URL in the expired article (which created a new mailnews window) and while it was processing it, I went back to the original MailNews window and clicked on an IMAP folder (in a different account, obviously). gdb says: #0 0x41b2a8d7 in nsMsgNewsFolder::RemoveMessage () from /usr/lib/mozilla/components/libmsgnews.so #1 0x41b08329 in nsNNTPArticleList::AddArticleKey () from /usr/lib/mozilla/components/libmsgnews.so #2 0x41b1a8f1 in nsNNTPProtocol::SendListGroupResponse () from /usr/lib/mozilla/components/libmsgnews.so #3 0x41b1b1b7 in nsNNTPProtocol::ProcessProtocolState () from /usr/lib/mozilla/components/libmsgnews.so #4 0x418c50c1 in nsMsgProtocol::OnDataAvailable () from /usr/lib/libmsgbaseutil.so #5 0x40743f71 in nsOnDataAvailableEvent::HandleEvent () from /usr/lib/mozilla/components/libnecko.so #6 0x407326db in nsARequestObserverEvent::HandlePLEvent () from /usr/lib/mozilla/components/libnecko.so #7 0x4017dd1f in PL_HandleEvent () from /usr/lib/libxpcom.so #8 0x4017e16c in PL_ProcessEventsBeforeID () from /usr/lib/libxpcom.so #9 0x411aae69 in processQueue () from /usr/lib/mozilla/components/libwidget_gtk.so #10 0x4014e37d in nsVoidArray::EnumerateForwards () from /usr/lib/libxpcom.so #11 0x411aaea7 in nsAppShell::ProcessBeforeID () from /usr/lib/mozilla/components/libwidget_gtk.so #12 0x411b1f9d in handle_gdk_event () from /usr/lib/mozilla/components/libwidget_gtk.so #13 0x40365d7f in gdk_event_dispatch () from /usr/lib/libgdk-1.2.so.0 #14 0x40399773 in g_main_dispatch () from /usr/lib/libglib-1.2.so.0 #15 0x40399d39 in g_main_iterate () from /usr/lib/libglib-1.2.so.0 #16 0x40399eec in g_main_run () from /usr/lib/libglib-1.2.so.0 #17 0x402b4333 in gtk_main () from /usr/lib/libgtk-1.2.so.0 #18 0x411aabd6 in nsAppShell::Run () from /usr/lib/mozilla/components/libwidget_gtk.so #19 0x4118a9f6 in nsAppShellService::Run () from /usr/lib/mozilla/components/libnsappshell.so #20 0x08051ea1 in DoCommandLines () #21 0x08052763 in main () #22 0x404f7627 in __libc_start_main (main=0x8052618 <main>, argc=1, ubp_av=0xbffff8c4, init=0x804ca2c <_init>, fini=0x8053ddc <_fini>, rtld_fini=0x4000dcc4 <_dl_fini>, stack_end=0xbffff8bc) at ../sysdeps/generic/libc-start.c:129
Reporter | ||
Comment 1•22 years ago
|
||
P.S. BuildID 2002033118 on RedHat Linux 7.2+updates.
Keywords: crash
Assignee | ||
Comment 2•22 years ago
|
||
Stephen/Seth, this is a dup, right?
I haven't experienced a crash, not seen a bug filed with that particular stack yet, so I'd have to say no.
QA Contact: gayatri → stephend
Assignee | ||
Comment 4•22 years ago
|
||
oh, yes, I see what probably happened - we probably closed the db out from under the news folder when you clicked on a different folder.
Component: Mail Database → Networking - News
Assignee | ||
Comment 5•22 years ago
|
||
call GetDatabase to make sure the db is open.
Comment 6•22 years ago
|
||
Comment on attachment 77293 [details] [diff] [review] proposed fix sr=sspitzer
Attachment #77293 -
Flags: superreview+
Updated•22 years ago
|
Attachment #77293 -
Flags: superreview+ → review+
Comment 7•22 years ago
|
||
Comment on attachment 77293 [details] [diff] [review] proposed fix r=naving
Updated•22 years ago
|
Target Milestone: --- → mozilla1.0
Assignee | ||
Comment 8•22 years ago
|
||
Nominating, it's a crash, and trivial to fix.
Comment 9•22 years ago
|
||
Marking [adt3] because I don't think this scenario is too likely, but since it's a crash and a pretty simple fix, I think we should take it.
Comment 11•22 years ago
|
||
adt1.0.0+ (on ADT's behalf) approval for checkin to 1.0. Fixing crashers improve our stability story.
Assignee | ||
Comment 12•22 years ago
|
||
Comment on attachment 77293 [details] [diff] [review] proposed fix check sr box for Seth.
Attachment #77293 -
Flags: superreview+
Comment 13•22 years ago
|
||
Comment on attachment 77293 [details] [diff] [review] proposed fix a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #77293 -
Flags: approval+
Assignee | ||
Comment 14•22 years ago
|
||
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
I've tried reproducing this with 2002-04-08-08 and a newsgroup that expires articles quickly using the testcase in comment #1. I can no longer reproduce this on Linux, Mac OS X and Windows 2000. For the trunk, this was the only incident (well, 2 of them, but it was the same reporter): http://climate/reports/searchstacksignature.cfm?stacksig=nsMsgNewsFolder%3A%3ARemoveMessage 2002032610 MozillaTrunk Windows NT 5.0 build 2195 2002-04-02 03:38:03 nsMsgNewsFolder::RemoveMessage 521aa5dd All of the other crashers with this stack were with the MozillaBranch. Verified FIXED
Status: RESOLVED → VERIFIED
Adding verified1.0.0 to get off queries.
Keywords: verified1.0.0
Updated•20 years ago
|
Product: MailNews → Core
Updated•15 years ago
|
Product: Core → MailNews Core
Updated•9 years ago
|
Blocks: dialer-rtl
Updated•9 years ago
|
No longer blocks: dialer-rtl
You need to log in
before you can comment on or make changes to this bug.
Description
•