4748 - Selecting Bookmarks | Personal | Tinderbox or Bonsai crashes

Status: VERIFIED FIXED

(Core Graveyard :: Tracking, defect, P1)

Description

[rubydoo123]

build: 1999040711
platform: win95, mac

expected: to be directed to the appropriate web page

what happens: selecting the option crashes the app.

APPRUNNER caused an invalid page fault in
module <unknown> at 00de:00000013.

talkback tracking ID DGV58MWZ

Comment 1

[don]

Re-assinged to [e-mail], set target milstone to M4, changed priority
to P1, and platform and OS to All.

Steve, see if you can figure out what's happening with all the crashing menu
items.

Comment 2

[Steve Lamm]

These bookmarks are loaded by setting the frame href,

window.frames[0].location.href

The url bar uses the browser app cores loadUrl() function. As a work around, we
could have the bookmarks call that instead. We should still find out why it is
crashing though.

Comment 3

[Steve Lamm]

*** Bug 4065 has been marked as a duplicate of this bug. ***

Comment 4

[Steve Lamm]

This is a regression from m3. waterson looked at this with me on Linux. It's
down in the old netlib. The mwcontext ends up null for some reason and then it's
dereferenced. Since no one really owns the old netlib, I am reassigning this to
jevering as suggested by chofmann.

Here's a sample stack trace. The code seems to crash in a different place
everytime.

#0  0x40176908 in net_ProcessHTTP (ce=0x830d880)
    at /export/slamm/gecko/mozilla/network/protocol/http/mkhttp.c:3720
#1  0x40241fbb in NET_ProcessNet (ready_fd=0x81050d8, fd_type=2)
    at /export/slamm/gecko/mozilla/network/main/mkgeturl.c:3355
#2  0x40249f49 in NET_PollSockets ()
    at /export/slamm/gecko/mozilla/network/main/mkselect.c:320
#3  0x4026ad62 in nsNetlibService::NetPollSocketsCallback (aTimer=0x83d05a8,
    aClosure=0x80f83f0)
    at /export/slamm/gecko/mozilla/network/module/nsNetService.cpp:1277
#4  0x40154de9 in TimerImpl::FireTimeout (this=0x83d05a8)
    at /export/slamm/gecko/mozilla/base/src/gtk/nsTimer.cpp:73
#5  0x401552d2 in nsTimerExpired (aCallData=0x83d05a8)
    at /export/slamm/gecko/mozilla/base/src/gtk/nsTimer.cpp:189
#6  0x40a59990 in g_timeout_dispatch (source_data=0x8218658,
    current_time=0xbffff410, user_data=0x83d05a8) at gmain.c:1147
#7  0x40a58c83 in g_main_dispatch (current_time=0xbffff410) at gmain.c:647
#8  0x40a5920f in g_main_iterate (block=1, dispatch=1) at gmain.c:854
#9  0x40a59391 in g_main_run (loop=0x817acb8) at gmain.c:912
#10 0x4098644b in gtk_main () at gtkmain.c:475
#11 0x400911d8 in nsAppShell::Run (this=0x813a1f8)
    at /export/slamm/gecko/mozilla/widget/src/gtk/nsAppShell.cpp:208
#12 0x40017e15 in nsAppShellService::Run (this=0x80da0e0)
    at /export/slamm/gecko/mozilla/xpfe/appshell/src/nsAppShellService.cpp:186
#13 0x804a8cc in main (argc=1, argv=0xbffff530)
    at /export/slamm/gecko/mozilla/xpfe/bootstrap/nsAppRunner.cpp:337

Comment 5

[Chris McAfee]

Adding netlib people.

Comment 6

[Claudius Gayle]

I just looked at MacOS1999041112, Linux1999041108, and WinNT 1999041010, or all
the Sunday4/11 builds and this seems to check out fine.
If it were marked fixed i'd mark it VERIFIED, but it's not. Since it's an M4 I'll
leave it on the radar.
cc'ing chofmann

Comment 7

[chris hofmann]

move to m5 to investigate why it now myseriously works.

Comment 8

[Claudius Gayle]

marking VERIFIED per above comments.

Comment 9

[leger]

Moving all Apprunner bugs past and present to Other component temporarily whilst
don and I set correct component.  Apprunner component will be deleted/retired
shortly.