Closed Bug 1518948 Opened 8 months ago Closed 7 months ago

Wrong loadInfo check in CSPService::AsyncOnChannelRedirect

Categories

(Core :: DOM: Security, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla66
Tracking Status
firefox66 --- fixed

People

(Reporter: baku, Assigned: baku)

Details

(Whiteboard: [domsecurity-backlog1] [domsecurity-active])

Attachments

(1 file)

Attached patch loadInfo.patchSplinter Review
Attachment #9035476 - Flags: review?(ckerschb)
Status: NEW → ASSIGNED
Whiteboard: [domsecurity-backlog1] [domsecurity-active]
Comment on attachment 9035476 [details] [diff] [review]
loadInfo.patch

Review of attachment 9035476 [details] [diff] [review]:
-----------------------------------------------------------------

thanks, r=me!
Attachment #9035476 - Flags: review?(ckerschb) → review+

Btw, did this cause any crashes? Because if it did we need to fine the source of the problem!

Flags: needinfo?(amarchesini)

Btw, did this cause any crashes? Because if it did we need to fine the source of the problem!

No crashes. Just bad code.

Flags: needinfo?(amarchesini)
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/50f5048d3b93
Wrong loadInfo check in CSPService::AsyncOnChannelRedirect, r=ckerschb
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in before you can comment on or make changes to this bug.