Closed
Bug 57187
Opened 25 years ago
Closed 25 years ago
reordering pointer assignment
Categories
(Core :: JavaScript Engine, defect, P3)
Tracking
()
VERIFIED
FIXED
mozilla0.8.1
People
(Reporter: dolafson, Assigned: rogerl)
Details
Attachments
(1 file)
|
417 bytes,
patch
|
Details | Diff | Splinter Review |
in jsnum.c function js_strtod, i = estr - cstr is calculated after free(cstr).
boundschecker marks this as an error. putting the free() directly after the
assignment makes boundschecker happy and many misreported leaks disappear
The code in question is here
http://lxr.mozilla.org/seamonkey/source/js/src/jsnum.c#824
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
||
Updated•25 years ago
|
Target Milestone: --- → mozilla0.8.1
|
|
||
Comment 2•25 years ago
|
||
Marking 0.8.1.
|
|
Assignee | |
Comment 3•25 years ago
|
||
|
|
Assignee | |
Comment 4•25 years ago
|
||
Another simple patch dying for a trivial review.
Status: NEW → ASSIGNED
|
|
||
Comment 5•25 years ago
|
||
sr=brendan@mozilla.org. I'm sure patrick or shaver would bless this change in a
trice; why not put r=beard and roll them dice?
/be
|
||
Comment 6•25 years ago
|
||
r=shaver.
|
|
Assignee | |
Comment 7•25 years ago
|
||
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•