278153 - crash @ nsDOMEventRTTearoff::GetEventReceiver()

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

GetEventReceiver crashes if the listener_manager is null.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Attachment: v1

I think nsGenericElement::GetListenerManager shouldn't return NS_OK if
it the aResults is nsnull.

The crash happens here:
http://lxr.mozilla.org/seamonkey/source/content/base/src/nsGenericElement.cpp#556


http://talkback-public.mozilla.org/talkback/fastfind.jsp?search=1&searchby=stacksig&match=contains&searchfor=GetEventReceiver&vendor=All&product=All&platform=All&buildid=&sdate=&stime=&edate=&etime=&sortby=bbid

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 171079 [details] [diff] [review]
v1

bz, what you think?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 171079 [details] [diff] [review]
v1

r+sr=bzbarsky, but with NS_ERROR_NOT_AVAILABLE instead, perhaps?

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Attachment: with NS_ERROR_NOT_AVAILABLE

Comment 5

[Allan Beaufour]

Checked in.