Last Comment Bug 676240 - Crash [@ nsXULTreeAccessible::GetChildCount()
: Crash [@ nsXULTreeAccessible::GetChildCount()
Status: RESOLVED FIXED
: crash
Product: Core
Classification: Components
Component: Disability Access APIs (show other bugs)
: 5 Branch
: x86 Windows XP
: -- critical (vote)
: mozilla8
Assigned To: alexander :surkov
:
: alexander :surkov
Mentors:
: 682067 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-03 07:04 PDT by jon
Modified: 2011-10-10 09:29 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (747 bytes, patch)
2011-08-03 11:16 PDT, alexander :surkov
tbsaunde+mozbugs: review+
Details | Diff | Splinter Review

Description jon 2011-08-03 07:04:43 PDT
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:5.0) Gecko/20100101 Firefox/5.0
Build ID: 20110615151330

Steps to reproduce:

my firefox 5.0 crashes on random websites and i need to restart it over and over


Actual results:

browsesing to a website


Expected results:

it shouldn't crash all day long
Comment 1 jon 2011-08-03 07:05:28 PDT
i have submitted bug after bug after bug and it doesn't seem like anything is happening
Comment 2 Robert Longson 2011-08-03 07:21:41 PDT
Please provide the crash report IDs from about:crashes from some of your recently submitted crashes.
Comment 4 Thomas Ahlblom 2011-08-03 08:22:16 PDT
0 	xul.dll 	nsXULTreeAccessible::GetChildCount 	accessible/src/xul/nsXULTreeAccessible.cpp:452
1 	xul.dll 	nsAccessibleWrap::get_accChildCount 	accessible/src/msaa/nsAccessibleWrap.cpp:239
2 	rpcrt4.dll 	Invoke 	
3 	rpcrt4.dll 	NdrStubCall2 	
4 	rpcrt4.dll 	NdrClientMapCommFault 	
5 	oleaut32.dll 	oleaut32.dll@0x129ae 	
6 	ole32.dll 	SyncStubInvoke 	
7 	ole32.dll 	StubInvoke 	
8 	ole32.dll 	IFillLockBytes_FillAppend_Proxy 	
9 	ole32.dll 	COIDHashTable::Compare
Comment 5 (mostly gone) XtC4UaLL [:xtc4uall] 2011-08-03 09:25:10 PDT
Per http://bit.ly/p4W3hE this happens on Firefox 5/6/7/8 Branches.
Comment 6 alexander :surkov 2011-08-03 11:16:58 PDT
Created attachment 550434 [details] [diff] [review]
patch
Comment 7 Trevor Saunders (:tbsaunde) 2011-08-03 13:24:35 PDT
Comment on attachment 550434 [details] [diff] [review]
patch


>+  if (!pcountChildren)
>+    return E_INVALIDARG;
>   *pcountChildren = 0;

blank line after return please.
Comment 8 alexander :surkov 2011-08-03 20:43:00 PDT
(In reply to comment #7)

> >+  if (!pcountChildren)
> >+    return E_INVALIDARG;
> >   *pcountChildren = 0;
> 
> blank line after return please.

sure if you like
Comment 9 alexander :surkov 2011-08-03 21:03:53 PDT
inbound land http://hg.mozilla.org/integration/mozilla-inbound/rev/0e7f85aa8137
Comment 10 alexander :surkov 2011-08-04 02:32:27 PDT
landed http://hg.mozilla.org/mozilla-central/rev/0e7f85aa8137
Comment 11 Wayne Mery (:wsmwk, NI for questions) 2011-10-10 09:29:13 PDT
*** Bug 682067 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.