Closed Bug 1942025 Opened 1 year ago Closed 1 year ago

The CSP of about:privatebrowsing is too permissive

Categories

(Firefox :: Private Browsing, defect)

defect

Tracking

()

RESOLVED FIXED
136 Branch
Tracking Status
firefox-esr115 --- wontfix
firefox-esr128 --- wontfix
firefox134 --- wontfix
firefox135 --- wontfix
firefox136 + fixed

People

(Reporter: tschuster, Assigned: tschuster)

References

Details

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

Attachments

(1 file)

The about:privatebrowsing page uses the following CSP policy: default-src chrome: blob:; object-src 'none'. This means scripts/styles etc. can be loaded dynamically from a blob URL. I didn't immediately see where this page even uses blobs. All our default-src policy should only include chrome: and resource:.

It seems like we load the logo inside the search bar from a blob URL.

Assignee: nobody → tschuster
Status: NEW → ASSIGNED
Keywords: sec-want
Pushed by tschuster@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/32d761bfc411 Improve the about:privatebrowsing CSP. r=firefox-desktop-core-reviewers ,Gijs
Group: firefox-core-security → core-security-release
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch

The patch landed in nightly and beta is affected.
:tschuster, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox135 to wontfix.

For more information, please visit BugBot documentation.

Flags: needinfo?(tschuster)

I don't think we need to uplift this.

Flags: needinfo?(tschuster)
QA Whiteboard: [post-critsmash-triage]
Flags: qe-verify-
Whiteboard: [adv-main136-]
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: