Closed Bug 524236 Opened 15 years ago Closed 15 years ago

NJ merge: arm actually needs to check used-ness of call instruction in asm_call

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
status1.9.2 --- beta5-fixed

People

(Reporter: graydon, Assigned: graydon)

References

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

fix the bug
1.10 KB, patch
gal
: review+
Details | Diff | Splinter Review
Attached patch fix the bug — — Splinter Review 
Recent change to asm_call (bug 523221) introduced a bug of its own. The instruction for asm_call might be unused, making its reservation null.

This patch corrects it.

(located by your friendly lirasm --random fuzzer mode :)
Attachment #408146 - Flags: review?(gal)
http://hg.mozilla.org/tracemonkey/rev/39cdbb22e58c
Whiteboard: fixed-in-tracemonkey
Attachment #408146 - Flags: review?(gal) → review+
(In reply to comment #0)
>
> (located by your friendly lirasm --random fuzzer mode :)

Woo!
http://hg.mozilla.org/mozilla-central/rev/39cdbb22e58c
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Flags: blocking1.9.2+
These bugs landed after b4 was cut. Moving flag out.
status1.9.2: beta4-fixed → final-fixed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: