Closed Bug 121118 Opened 23 years ago Closed 23 years ago

xpti can crash if param type is not resolvable

Categories

(Core :: XPCOM, defect)

defect
Not set
critical

Tracking

()

VERIFIED FIXED

People

(Reporter: jband_mozilla, Assigned: jband_mozilla)

References

Details

(Keywords: crash)

Attachments

(1 file)

I made a bad assumption in xptiInterfaceEntry::GetEntryForParam that all params
that are interfaces types will be resolvable at runtime. I found that on some
interfaces this hit the assert in that code. Big suprise... People declare
params using types that are forward declared but never actually fully declared
in idl. This is perfectly legal. xpti clients should get a error result rather
than a crash. patch coming up.
Attached patch proposed fixSplinter Review
Severity: normal → critical
Keywords: crash
Comment on attachment 65902 [details] [diff] [review]
proposed fix

r=dbradley
Attachment #65902 - Flags: review+
*** Bug 122463 has been marked as a duplicate of this bug. ***
checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Marking Verified -
Status: RESOLVED → VERIFIED
Component: xpidl → XPCOM
QA Contact: pschwartau → xpcom
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: