Incorrect magic number check for ABC files.

VERIFIED FIXED

Status

VERIFIED FIXED
12 years ago
9 years ago

People

(Reporter: dnljms, Unassigned)

Tracking

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

12 years ago
User-Agent:       Opera/9.10 (X11; Linux i686; U; en)
Build Identifier: 

Running a linux build of Tamarin, the check for the ABC magic number is matching the executable.

Reproducible: Always

Steps to Reproduce:
1. Build Tamarin (using Ubuntu Feisty)
2. Run 'avmplus'
Actual Results:  
Error message:

VerifyError: Error #1042: Not an ABC file.  major_version=17996 minor_version=17791.


Expected Results:  
The usage message
(Reporter)

Comment 1

12 years ago
Created attachment 256230 [details] [diff] [review]
Fix the magic number check.

Comment 2

12 years ago
You'll need to request review on the patch from one of the Tamarin module owners or peers. http://www.mozilla.org/owners.html#tamarin
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Reporter)

Comment 3

12 years ago
Created attachment 256269 [details] [diff] [review]
Fix the magic number check

Oh sorry, I hadn't read the guide. I've also recreated the patch with more context.
Attachment #256230 - Attachment is obsolete: true
Attachment #256269 - Flags: review?(stejohns)

Updated

12 years ago
Attachment #256269 - Flags: review?(stejohns) → review+

Comment 4

11 years ago
Transfer Bug from Core->Tamarin to Tamarin Product.
Component: Tamarin → Virtual Machine
Product: Core → Tamarin
Version: Trunk → unspecified

Comment 5

10 years ago
Bug has already been submitted according to stejohns, marked as resolved.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

9 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.