Closed Bug 2166 Opened 26 years ago Closed 26 years ago

[PP] Copy (Command-C) & Select All (Command-A) crash viewer

Categories

(Core Graveyard :: Viewer App, defect, P1)

PowerPC
Mac System 8.5

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: elig, Assigned: mozeditor)

Details

* TITLE/SUMMARY [PP] Pressing Command-A three times in URL window will crash Viewer * STEPS TO REPRODUCE 0) Launch 1.4.98 build of Viewer 1) Click in the URL window, and press Command-A three or four times. * RESULT - What happened Content window re-draws on each Command-A. Then Viewer crashes. 100% reproducible. Stack crawl enclosed. Occurs if takes place while loading web page, or while no other threads are taking place. Only occurs with Command-A ( - What was expected No crash. Don't take me back to Kansas. ;) * REGRESSION - Occurs On viewer (1.4.98 build for Mac OS) - Doesn't Occur On viewer (1.4.98 build for Win32, using Control-A. Gives 'invalid key' beep.) * CONFIGURATIONS TESTED - PowerMac 8500/150 (233 Mhz 604e), 64 MB RAM, Mac OS 8.5.1 * OBLIGATORY STACK CRAWL Calling chain using A6/R1 links Back chain ISA Caller 00000000 PPC 0BA85E74 039F0CC0 PPC 0BA7D1D8 039F0C60 PPC 0BA7CBEC 039F0C20 PPC 0B754F18 nsMacMessageSink::DispatchMenuCommand(EventRecord&, long)+004FC 039F0B40 PPC 0B755300 nsMacMessageSink::DispatchMenuCommand(EventRecord&, long)+008E4 039F0AE0 PPC 0B75587C nsMacMessageSink::DispatchMenuCommand(EventRecord&, long)+00E60 039F0AA0 PPC 0B7558F0 nsMacMessageSink::DispatchMenuCommand(EventRecord&, long)+00ED4 039F0A50 PPC 0B755AA0 nsMacMessageSink::DispatchMenuCommand(EventRecord&, long)+01084 039F0A00 PPC 0B754A68 nsMacMessageSink::DispatchMenuCommand(EventRecord&, long)+0004C 039F09B0 PPC 0B752B04 NS_GetWidgetNativeData(nsISupports*, void**)+0BD54 039F0950 PPC 0B752D00 NS_GetWidgetNativeData(nsISupports*, void**)+0BF50 039F08E0 PPC 0B7422D0 039F08A0 PPC 0B742248 039F0850 PPC 0BA77E14 039F0800 PPC 0BA7D120 039F06B0 PPC 0BA7821C 039F05C0 PPC 0BA79090 039F0570 PPC 0B5B6798 NS_NewPostData(int, char*, nsIPostData**)+02DB0 039F04F0 PPC 0B6323F8 NS_NewImageDocument(nsIDocument**)+153DC 039F04B0 PPC 0B632394 NS_NewImageDocument(nsIDocument**)+15378 039F0470 PPC 0B6315BC NS_NewImageDocument(nsIDocument**)+145A0 039F0420 PPC 0B6277B8 NS_NewImageDocument(nsIDocument**)+0A79C 039F03D0 PPC 0B9FAF44 nsVoidArray::~nsVoidArray()+00028 039F0390 PPC 0BA6E53C __dla__FPv+00014 039F0330 PPC 0BA6E498 operator delete(void*)+00014 PowerPC illegal instruction at FFC10000 _AA6A+00DFE
Assignee: rickg → sdagley
Also, please note that just pressing Command-C will immediately bomb this build, too.
[Note: Unlike the Wizard of Oz, you may need to click your h/e/e/l/s/ Command-A key more than three times in order to be carried off into the whiteness (of the debugger.) Occurs using 11.4.98 debug build]
Assignee: sdagley → jfrancis
Status: NEW → ASSIGNED
Summary: [PP] Pressing Command-A three times in URL window will crash Viewer → [PP] Choosing Copy (Command-C) or Select All (Command-A) three times yields crash
[Please note that the same behavior occurs if you select these items from the menu; i.e. selecting "Copy" once will crash viewer, and selecting "Select All" a few times will also crash viewer.]
Summary: [PP] Choosing Copy (Command-C) or Select All (Command-A) three times yields crash → [PP] Copy (Command-C) & Select All (Command-A) crash viewer
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
copy seems to no longer be a problem (works for me on recent builds) but select all had badly broken addref/release code. fixed.
Status: RESOLVED → VERIFIED
Verified fixed in 1.20.99 build of Viewer. Specifically, tried lots'o'random permutations of the keyboard & mouse commands for cut, copy, paste & select all; despite lots'o'clicks, no crashes took place. I can only conclude that you must have fixed it, Joe. Oh, wait a sec...I've just picked up a fault in the 604e unit... it looks like it's going to go 400 Mhz within 72 hours... ;)
Inserting Milestone info.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.