Closed Bug 115527 Opened 23 years ago Closed 23 years ago

Trunk Crash when visiting webpages with redirections [@ nsTimerImpl::Process]

Categories

(SeaMonkey :: General, defect, P1)

Product:
SeaMonkey
Component:
General
Platform:
x86
All
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED
Milestone:
mozilla0.9.8

People

(Reporter: kryptolus, Assigned: pavlov)

References

(
URL
)

Details

(Keywords: crash, topcrash)

Crash Data

Attachments

(1 file, 1 obsolete file)

Real fix
3.46 KB, patch
bugs
: review+
mscott
: superreview+
Details | Diff | Splinter Review 
Mozilla 2001121606 crashes immediately upon entering this site.
Talkback id: TB555964W
Also crashing 20011216 on Win2k.
OS -> All.

another redirection crasher ?

Stephen, can you retreive Talkback data please ?
Severity: normal → critical
Keywords: crash
OS: Linux → All
You're right.
I used netscape 4 to go that address and then took the address after redirection
and pasted it to mozilla and everything was fine.
So whatever the reason for it crashing, it probably happens during redirection.
Sorry, that ID comes up empty.  cc: me if you get another, thanks.
Easy does it: TB557874Q
It crashes on linux too, 2001-121606
TB: TB558090X
(2001-121506: No crash)
*** Bug 115531 has been marked as a duplicate of this bug. ***
*** Bug 115529 has been marked as a duplicate of this bug. ***
Problem is with <meta http-equiv="refresh" content="1;url=url_here">

Whatever it is, it was caused by a checking yesterday
Updating summary and setting new as bug 115529
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: immediate crash at http://go.to/kg7faq → Crash when visiting webpages with redirections
It's timer code, and I think it might belong to pavlov@netscape.com.  Here's my
own stack.
Stack Signature  nsTimerImpl::Process 22202294
Trigger Time 2001-12-16 15:12:51
Email Address stephend@netscape.com
URL visited http://home.netscape.com
User Comments Crashed on a redirect
Build ID 2001121611
Product ID MozillaTrunk
Platform
Operating System Win32
Module
Trigger Reason Access violation
Stack Trace
nsTimerImpl::Process [d:\builds\seamonkey\mozilla\xpcom\threads\nsTimerImpl.cpp,
line 236]
handleMyEvent [d:\builds\seamonkey\mozilla\xpcom\threads\nsTimerImpl.cpp, line 267]
PL_HandleEvent [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line 591]
PL_ProcessPendingEvents [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c,
line 524]
_md_EventReceiverProc [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line
1072]
nsAppShellService::Run
[d:\builds\seamonkey\mozilla\xpfe\appshell\src\nsAppShellService.cpp, line 303]
netscp6.exe + 0x167e (0x0040167e)
netscp6.exe + 0x121c (0x0040121c)
netscp6.exe + 0x3222 (0x00403222)
KERNEL32.DLL + 0x17d08 (0x77e97d08)
Attached patch Fix (obsolete) — — Splinter Review 
this patch doesn't actually fix the crash in the url above, but the stack is
completly different.
the new timers weren't addrefing the nsITimerCallback interface that was passed
to it.	now it does.
So, for the actual crash as this bug was filed (http://go.to/kg7faq), here is my
stack:

Stack Signature  gkcontent.dll + 0xff81d (0x01b3f81d) f2e072ed
Trigger Time 2001-12-16 19:25:26
Email Address stephend@netscape.com
URL visited http://go.to/kg7faq
User Comments
Build ID 2001121611
Product ID MozillaTrunk
Platform
Operating System Win32
Module
Trigger Reason Access violation
Stack Trace
gkcontent.dll + 0xff81d (0x01b3f81d)
SETUPAPI.DLL + 0x30c24 (0x778b0c24) 
linux cvs, some 5h old, stack for http://go.to/kg7faq

#0  0x4014ae89 in nsTimerImpl::Process () from libxpcom.so
#1  0x4014aedf in handleMyEvent () from libxpcom.so
#2  0x40146fef in PL_HandleEvent () from libxpcom.so
#3  0x401473ff in PL_ProcessEventsBeforeID () from libxpcom.so
#4  0x406f7c43 in processQueue () from libwidget_gtk.so
#5  0x40117097 in nsVoidArray::EnumerateForwards () from libxpcom.so
#6  0x406f7c80 in nsAppShell::ProcessBeforeID () from libwidget_gtk.so
#7  0x406fe64e in handle_gdk_event () from libwidget_gtk.so
#8  0x4035316b in gdk_event_dispatch (source_data=0x0, current_time=0xbffff430,
user_data=0x0) at gdkevents.c:2139
#9  0x40384055 in g_main_dispatch (dispatch_time=0xbffff430) at gmain.c:656
#10 0x40384659 in g_main_iterate (block=1, dispatch=1) at gmain.c:877
#11 0x403847e8 in g_main_run (loop=0x81f4b30) at gmain.c:935
#12 0x4029865b in gtk_main () at gtkmain.c:524
#13 0x406f79c5 in nsAppShell::Run () from libwidget_gtk.so
#14 0x406da17e in nsAppShellService::Run () from libnsappshell.so
#15 0x080516db in main1 ()
#16 0x08052009 in main ()
#17 0x404cc627 in __libc_start_main (main=0x8051ed0 <main>, argc=1,
ubp_av=0xbffff834, init=0x804bf44 <_init>, fini=0x8052edc <_fini>,
rtld_fini=0x4000dcc4 <_dl_fini>, stack_end=0xbffff82c)
    at ../sysdeps/generic/libc-start.c:129
Attachment #61917 - Attachment is obsolete: true
*** Bug 115524 has been marked as a duplicate of this bug. ***
Attached patch Real fix — — Splinter Review 
This patch fixes the problems by addrefing/releasing the nsITimerCallback as
well as removing a useless addref/release of the timer and shuffling a release
back in to the timer so that it always gets released on the right thread.
Keywords: review
reassigning to self
Assignee: asa → pavlov
Priority: -- → P1
Target Milestone: --- → mozilla0.9.8
Comment on attachment 61927 [details] [diff] [review]
Real fix

sr=mscott
Attachment #61927 - Flags: superreview+
fixed
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
*** Bug 115595 has been marked as a duplicate of this bug. ***
*** Bug 115598 has been marked as a duplicate of this bug. ***
*** Bug 115615 has been marked as a duplicate of this bug. ***
*** Bug 115685 has been marked as a duplicate of this bug. ***
*** Bug 115781 has been marked as a duplicate of this bug. ***
Adding talkback keywords and stack so we can track/verify this one.
Keywords: topcrash
Summary: Crash when visiting webpages with redirections → Trunk Crash when visiting webpages with redirections [@ nsTimerImpl::Process]
*** Bug 115679 has been marked as a duplicate of this bug. ***
*** Bug 115578 has been marked as a duplicate of this bug. ***
*** Bug 115596 has been marked as a duplicate of this bug. ***
Verified Fixed.  Talkback data shows this last crashed on 12/17 (only 2 crashes
with that day's builds after seeing 388 crashes with 12/16 builds)
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
Crash Signature: [@ nsTimerImpl::Process]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: