Closed Bug 2031365 Opened 7 months ago Closed 6 months ago

Prevent implicit conversion of js_pod_arena_malloc size_t argument

Categories

(Core :: JavaScript Engine, defect, P1)

defect

Tracking

()

RESOLVED FIXED
152 Branch
Tracking Status
firefox-esr115 --- wontfix
firefox-esr140 --- wontfix
firefox150 --- wontfix
firefox151 --- wontfix
firefox152 --- fixed

People

(Reporter: abienner, Assigned: abienner)

References

(Blocks 1 open bug)

Details

(Keywords: sec-want, Whiteboard: [adv-main152-])

Attachments

(1 external file)

Follow up of bug 2028731.

This is not a security bug per say, but a follow up to limit the risk of having similar bugs in the future.

Since it might expose information about the original bug (bug 2028731), it should be considered as a security bug until bug 2028731 has been released.

Assignee: nobody → abienner
Status: NEW → ASSIGNED
Depends on: 2028731
Group: core-security → javascript-core-security
Keywords: sec-want
Attached file (secure) —
Blocks: sm-security
Severity: -- → S4
Priority: -- → P1
Type: enhancement → defect

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:abienner, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Flags: needinfo?(amarchesini)
Flags: needinfo?(abienner)

It's expected: it's a follow up (test) for a sec-high which has been landed recently (bug 2028731). I'm waiting for the original fix to have been live for 4 weeks before landing that one, as requested by our procedure for security bug.
"Tests should only be checked in later, after an official Firefox release that contains the fix has been live for at least four weeks."

Flags: needinfo?(amarchesini)
Flags: needinfo?(abienner)
Pushed by abienner@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/d03d9d86efc5 https://hg.mozilla.org/integration/autoland/rev/8e0d2cdf22f8 Prevent implicit conversion of size_t arguments when calling js_pod alloc functions r=baku,spidermonkey-reviewers,sfink
Group: javascript-core-security → core-security-release
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
QA Whiteboard: [sec] [qa-triage-done-c153/b152]
Whiteboard: [adv-main-152-]
Whiteboard: [adv-main-152-] → [adv-main152-]
Group: core-security-release
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: