Jim Dunn Assignee
	Description • 25 years ago

Clicking on Messenger causes the AIX build to crash

	Jim Dunn Assignee
	Comment 1 • 25 years ago

candidate for beta1...

	lchiang
	Comment 2 • 25 years ago

clearing my name as QA contact to nobody@mozilla.org since I'm only testing on 
Linux, Mac, and Win32.

	leger
	Comment 3 • 25 years ago

Putting on PDT- radar for beta1.

	leger
	Comment 4 • 25 years ago

Adding "crash" keyword to all known open crasher bugs.

	Jim Dunn Assignee
	Comment 5 • 25 years ago

traceback

Segmentation fault in ptrgl._ptrgl
[/builds/jdunn/5.0/mozilla/dist/bin/components/libxpconnect.so] at 0xd14377e8
($t1)
0xd14377e8 (_ptrgl)    800b0000        lwz   r0,0x0(r11)
(dbx) t
ptrgl._ptrgl() at 0xd14377e8
IsWrappedJS(nsISupports*)(0x204c3718), line 235 in "xpcwrappedjsclass.cpp"
NativeInterface2JSObject(JSContext*,nsIXPConnectJSObjectHolder**,nsISupports*,const
nsID*,JSObject*,unsigned int*)(0x209bc998, 0x2ff1f650, 0x204c3718, 0x20d40c28,
0x20cea170, 0x2ff1f864), line 652 in "xpcconvert.cpp"
unnamed block $b11366, line 330 in "xpcconvert.cpp"
unnamed block $b11365, line 330 in "xpcconvert.cpp"
NativeData2JS(JSContext*,long*,const void*,const nsXPTType&,const
nsID*,JSObject*,unsigned int*)(0x209bc998, 0x2ff1f770, 0x2ff1f7d0, 0x2ff1f778,
0x20d40c28, 0x20cea170, 0x2ff1f864), line 330 in "xpcconvert.cpp"
unnamed block $b11525, line 982 in "xpcwrappednativeclass.cpp"
CallWrappedMethod(JSContext*,nsXPCWrappedNative*,const
XPCNativeMemberDescriptor*,nsXPCWrappedNativeClass::CallMode,unsigned
int,long*,long*)(0x20d40568, 0x209bc998, 0x20d40a08, 0x20d405c8, 0x0, 0x1,
0x20a40374, 0x2ff1f9f4), line 982 in "xpcwrappednativeclass.cpp"
WrappedNative_CallMethod(JSContext*,JSObject*,unsigned
int,long*,long*)(0x209bc998, 0x20cea170, 0x1, 0x20a40374, 0x2ff1f9f4), line 198
in "xpcwrappednativejsops.cpp"
js_Invoke(0x209bc998, 0x1, 0x0), line 666 in "jsinterp.c"
js_Interpret(0x209bc998, 0x2ff20250), line 2262 in "jsinterp.c"
js_Invoke(0x209bc998, 0x0, 0x0), line 682 in "jsinterp.c"
js_Interpret(0x209bc998, 0x2ff20a70), line 2262 in "jsinterp.c"
js_Invoke(0x209bc998, 0x0, 0x0), line 682 in "jsinterp.c"
js_Interpret(0x209bc998, 0x2ff21290), line 2262 in "jsinterp.c"
js_Invoke(0x209bc998, 0x1, 0x2), line 682 in "jsinterp.c"
js_InternalInvoke(0x209bc998, 0x2070cb58, 0x2070c1a8, 0x0, 0x1, 0x2ff21504,
0x2ff21498), line 759 in "jsinterp.c"
JS_CallFunctionValue(0x209bc998, 0x2070cb58, 0x2070c1a8, 0x1, 0x2ff21504,
0x2ff21498), line 2772 in "jsapi.c"
CallEventHandler(void*,void*,unsigned int,void*,int*)(0x20226418, 0x2070cb58,
0x2070c1a8, 0x1, 0x2ff21504, 0x2ff215c4), line 561 in "nsJSEnvironment.cpp"
HandleEvent(nsIDOMEvent*)(0x20b505c8, 0x20c7da9c), line 128 in
"nsJSEventListener.cpp"
HandleEventSubType(nsListenerStruct*,nsIDOMEvent*,unsigned int)(0x20b504a8,
0x20b50b48, 0x20c7da9c, 0x1), line 677 in "nsEventListenerManager.cpp"
unnamed block $b2628, line 1228 in "nsEventListenerManager.cpp"
unnamed block $b2627, line 1228 in "nsEventListenerManager.cpp"
unnamed block $b2626, line 1228 in "nsEventListenerManager.cpp"
HandleEvent(nsIPresContext*,nsEvent*,nsIDOMEvent**,unsigned
int,nsEventStatus*)(0x20b504a8, 0x209b97b8, 0x2ff21a38, 0x2ff219bc, 0x7,
0x2ff21a30), line 1228 in "nsEventListenerManager.cpp"
HandleDOMEvent(nsIPresContext*,nsEvent*,nsIDOMEvent**,unsigned
int,nsEventStatus*)(0x209bc908, 0x209b97b8, 0x2ff21a38, 0x2ff219bc, 0x1,
0x2ff21a30), line 3327 in "nsGlobalWindow.cpp"
unnamed block $b10187, line 3156 in "nsWebShell.cpp"
unnamed block $b10186, line 3156 in "nsWebShell.cpp"
unnamed block $b10185, line 3156 in "nsWebShell.cpp"
unnamed block $b10184, line 3156 in "nsWebShell.cpp"
OnEndDocumentLoad(nsIDocumentLoader*,nsIChannel*,unsigned int)(0x209baf38,
0x209bb158, 0x209e39a8, 0x0), line 3156 in "nsWebShell.cpp"
unnamed block $b10071, line 602 in "nsDocLoader.cpp"
FireOnEndDocumentLoad(nsDocLoaderImpl*,nsIChannel*,unsigned int)(0x209bb158,
0x209bb158, 0x209e39a8, 0x0), line 602 in "nsDocLoader.cpp"
unnamed block $b10068, line 492 in "nsDocLoader.cpp"
unnamed block $b10067, line 492 in "nsDocLoader.cpp"
DocLoaderIsEmpty(unsigned int)(0x209bb158, 0x0), line 492 in "nsDocLoader.cpp"
unnamed block $b10066, line 437 in "nsDocLoader.cpp"
OnStopRequest(nsIChannel*,nsISupports*,unsigned int,const unsigned
short*)(0x209bb158, 0x20ccee98, 0x0, 0x0, 0x0), line 437 in "nsDocLoader.cpp"
RemoveChannel(nsIChannel*,nsISupports*,unsigned int,const unsigned
short*)(0x209bb1b8, 0x20ccee98, 0x0, 0x0, 0x0), line 535 in "nsLoadGroup.cpp"
OnStopRequest(nsIChannel*,nsISupports*,unsigned int,const unsigned
short*)(0x20ccee98, 0x20ccef48, 0x0, 0x0, 0x0), line 450 in "nsFileChannel.cpp"
HandleEvent()(0x20cd1198), line 278 in "nsAsyncStreamListener.cpp"
HandlePLEvent(PLEvent*)(0x20cd07b8), line 93 in "nsAsyncStreamListener.cpp"
PL_HandleEvent(0x20cd07b8), line 526 in "plevent.c"
unnamed block $b16182, line 487 in "plevent.c"
PL_ProcessPendingEvents(0x20079398), line 487 in "plevent.c"
ProcessPendingEvents()(0x20079348), line 228 in "nsEventQueue.cpp"
event_processor_callback(void*,int,GdkInputCondition)(0x20079348, 0x5, 0x1),
line 141 in "nsAppShell.cpp"
our_gdk_io_invoke(_GIOChannel*,GIOCondition,void*)(0x20464d08, 0x1, 0x200e2eb8),
line 53 in "nsAppShell.cpp"
g_io_unix_dispatch(0x20464d28, 0x2ff226e8, 0x200e2eb8), line 135 in "giounix.c"
unnamed block $b12508, line 656 in "gmain.c"
unnamed block $b12507, line 656 in "gmain.c"
g_main_dispatch(0x2ff226e8), line 656 in "gmain.c"
g_main_iterate(0x1, 0x1), line 874 in "gmain.c"
g_main_run(0x200e31c8), line 932 in "gmain.c"
gtk_main(), line 476 in "gtkmain.c"
Run()(0x20145238), line 304 in "nsAppShell.cpp"
Run()(0x20079058), line 455 in "nsAppShellService.cpp"
main1(int,char**)(argc = 1, argv = 0x2ff22af0), line 597 in "nsAppRunner.cpp"
main(argc = 1, argv = 0x2ff22af0), line 690 in "nsAppRunner.cpp"

	Jim Dunn Assignee
	Comment 6 • 25 years ago

changing QA contact, cuz hong don't care

	Jim Dunn Assignee
	Comment 7 • 25 years ago

changing to M15

	Jim Dunn Assignee
	Comment 8 • 25 years ago

changing to M16

	Mike
	Comment 9 • 24 years ago

M16 has been out, moving to M17.  Someone should check in on that, there 200+ 
bugs still marked for M16

	Jim Dunn Assignee
	Comment 11 • 23 years ago

updating... (adding Mary)

	Jim Dunn Assignee
	Comment 12 • 23 years ago

submitted by Mary Hoetzel (IBM)
r=jdunn@netscape.com

	Jim Dunn Assignee
	Comment 13 • 23 years ago

Jim, since you're the resident AIX expert and you approved the patch, I 
think it's safe to ship this one straight to drivers for a=.

- cls

	Jim Dunn Assignee
	Comment 14 • 23 years ago

fix checked in... thanks everyone!

	Jim Dunn Assignee
	Comment 15 • 23 years ago

oops... I also got this...
sr/a=blizzard

	Jim Dunn Assignee
	Comment 16 • 23 years ago

I want to re-open this bug and back out the fix.
The fix doesn't work in all cases and assumes single
inheritence.

I would like to revert back to revision 1.3
Here is the diff from version 1.4.

Index: xptcinvoke_asm_ppc_aix.s
===================================================================
RCS file:
/cvsroot/mozilla/xpcom/reflect/xptcall/src/md/unix/xptcinvoke_asm_ppc_aix.s,v
retrieving revision 1.4
diff -u -r1.4 xptcinvoke_asm_ppc_aix.s
--- xptcinvoke_asm_ppc_aix.s    2001/06/15 12:19:18     1.4
+++ xptcinvoke_asm_ppc_aix.s    2001/11/07 13:36:34
@@ -114,7 +114,8 @@
                addi    r5,r5,8                 # step over junk at start of
vTable !
                lwzx    r11,r5,r4               # get function pointer
 
-               lwz     r5,12(r4)               # Value at r4(vtable) + offset
12 is used to adjust r3 ('that' pointer)
+               addi    r5,r5,-4                # We need to manually adjust the
'that' pointer, this is CFRONT based
+               lwzx    r5,r4,r5                # offset = r4(vtable) +
r5(methodIndex offset) - 4 
                add     r3,r5,r3                # adjust 'that' r3 = r3 + r5
 
                lwz     r4,28(sp)

	Jim Dunn Assignee
	Comment 17 • 23 years ago

adding cls, looking for a r=.
THIS IS AIX only.

	Jim Dunn Assignee
	Comment 18 • 23 years ago

adding phil

	hacker formerly known as seawood@netscape.com
	Comment 19 • 23 years ago

Umm, sure. r=cls

	Jim Dunn Assignee
	Comment 20 • 23 years ago

fix checked in

	Christopher Blizzard (:blizzard)
	Comment 21 • 23 years ago

Did this get checked into the 0.9.6 branch?

	Jim Dunn Assignee
	Comment 22 • 23 years ago

yes it did...