717447 - Crash loading JSM with `catch ({})`

Status: RESOLVED DUPLICATE of bug 715561

(Core :: JavaScript Engine, defect)

Description

[satyr]

Attachment: try {} catch ({}) {}

Importing the attached JSM (e.g.: `Components.utils.import('file:///path/to/emptycatch.jsm')`) crashes Nightly.

UA: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:12.0a1) Gecko/20120111 Firefox/12.0a1

Comment 1

[Boris Zbarsky [:bzbarsky]]

Is this a regression?  If so, any idea when this regressed?

Comment 2

[satyr]

(In reply to Boris Zbarsky (:bz) from comment #1)
> Is this a regression?

At least latest Aurora doesn't have this issue.

> If so, any idea when this regressed?

Found it on Xmas via <https://github.com/satyr/uc/issues/3>.

Comment 3

[Sean Newman]

Yeah, I've reported it as soon as the crashes appeared for me, so it was 25 of Dec.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Would you be willing to use http://harthur.github.com/mozregression/ to pin down when it appeared?

Comment 5

[satyr]

(In reply to Boris Zbarsky (:bz) from comment #4)

Last good nightly: 2011-12-23
First bad nightly: 2011-12-24

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2011-12-23&enddate=2011-12-24

Comment 6

[Scoobidiver (away)]

(In reply to satyr from comment #5)
> Pushlog:
> http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2011-12-
> 23&enddate=2011-12-24
Nightlies are not built at midnight but around 3AM although it works in this case. The regression range is:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=c5b90ea7e475&tochange=f63a99195987

Comment 7

[Boris Zbarsky [:bzbarsky]]

Luke, you seem to have some changes in that range?

Comment 8

[Luke Wagner [:luke]]

Ah, yes.  I am fairly certain this is a dup of bug 715561 (empty destructuring in a context that could cause XDR).  Sorry for the trouble.

satyr: do you suppose you could verify the fix?  The fix landed today so isn't in nightly yet, but here is the the OS X 10.6 dmg: http://stage.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-inbound-macosx64/1326404603/firefox-12.0a1.en-US.mac.dmg

Comment 9

[satyr]

(In reply to Luke Wagner [:luke] from comment #8)
> satyr: do you suppose you could verify the fix?  The fix landed today so
> isn't in nightly yet, but here is the the OS X 10.6 dmg:
> http://stage.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-
> inbound-macosx64/1326404603/firefox-12.0a1.en-US.mac.dmg

Verified.

Comment 10

[Luke Wagner [:luke]]

Thank you for your help!