[ActiveX] iehtmlelement all property call crashes

RESOLVED FIXED

Status

Core Graveyard
Embedding: ActiveX Wrapper
--
critical
RESOLVED FIXED
14 years ago
6 years ago

People

(Reporter: Alexandre Trémon, Assigned: Adam Lock)

Tracking

Trunk
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

14 years ago
User-Agent:       
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; fr-FR; rv:1.6) Gecko/20040113

Crash in:
http://lxr.mozilla.org/seamonkey/source/embedding/browser/activex/src/common/IEHtmlElementCollection.cpp#159

Reproducible: Always
Steps to Reproduce:
VB exemple:

Dim document As HTMLDocument
Dim coll As Object
Set document = MozillaBrowser1.document
Set coll = document.body.all
(Reporter)

Comment 1

14 years ago
Created attachment 140734 [details] [diff] [review]
Changes in PopulateFromDOMNode method.

CIEHtmlElement::get_all method calls
CIEHtmlElementCollection::CreateFromParentNode with bRecurseChildren argument
set to true.
This in turn calls PopulateFromDOMNode with bRecurseChildren argument set to
true.
(Reporter)

Comment 2

14 years ago
Created attachment 140735 [details] [diff] [review]
did it again (tab indentation instead of spaces)
Attachment #140734 - Attachment is obsolete: true
(Reporter)

Comment 3

14 years ago
Created attachment 140736 [details] [diff] [review]
indentation!
Attachment #140735 - Attachment is obsolete: true
(Assignee)

Comment 4

14 years ago
Comment on attachment 140736 [details] [diff] [review]
indentation!

r=adamlock
Attachment #140736 - Flags: review+
(Reporter)

Updated

14 years ago
Attachment #140736 - Flags: superreview?(jst)
Comment on attachment 140736 [details] [diff] [review]
indentation!

sr=jst
Attachment #140736 - Flags: superreview?(jst) → superreview+
(Assignee)

Comment 6

14 years ago
Fix is checked in
Status: UNCONFIRMED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Component: Embedding: ActiveX Wrapper → Embedding: ActiveX Wrapper
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.