Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't adopt result of NSURLSession.sessionWithConfiguration #19393

Conversation

achristensen07
Copy link
Contributor

@achristensen07 achristensen07 commented Oct 21, 2023

bfc8bf8

Don't adopt result of NSURLSession.sessionWithConfiguration
https://bugs.webkit.org/show_bug.cgi?id=263501

Reviewed by Chris Dumez.

If it were ever released, it would cause memory corruption.
It's not, but still not good practice.

* Source/WebKit/NetworkProcess/PrivateClickMeasurement/cocoa/PrivateClickMeasurementNetworkLoaderCocoa.mm:
(WebKit::PCM::statelessSessionWithoutRedirects):

Canonical link: https://commits.webkit.org/269631@main

cb70fad

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  wincairo
βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug   πŸ§ͺ wpe-wk2
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2 βœ… πŸ§ͺ api-mac βœ… πŸ›  gtk
  πŸ§ͺ ios-wk2-wpt   πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios   πŸ§ͺ mac-wk2 βœ… πŸ§ͺ api-gtk
βœ… πŸ›  tv βœ… πŸ§ͺ mac-AS-debug-wk2
βœ… πŸ›  tv-sim
βœ… πŸ›  watch
βœ… πŸ›  πŸ§ͺ unsafe-merge βœ… πŸ›  watch-sim

@achristensen07 achristensen07 self-assigned this Oct 21, 2023
@achristensen07 achristensen07 added the WebKit Misc. For miscellaneous bugs in the WebKit framework (and not JavaScriptCore or WebCore). label Oct 21, 2023
Copy link
Contributor

@cdumez cdumez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ouch

@achristensen07 achristensen07 added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Oct 21, 2023
https://bugs.webkit.org/show_bug.cgi?id=263501

Reviewed by Chris Dumez.

If it were ever released, it would cause memory corruption.
It's not, but still not good practice.

* Source/WebKit/NetworkProcess/PrivateClickMeasurement/cocoa/PrivateClickMeasurementNetworkLoaderCocoa.mm:
(WebKit::PCM::statelessSessionWithoutRedirects):

Canonical link: https://commits.webkit.org/269631@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Dont-adopt-result-of-NSURLSession-sessionWithConfiguration branch from cb70fad to bfc8bf8 Compare October 21, 2023 21:18
@webkit-commit-queue
Copy link
Collaborator

Committed 269631@main (bfc8bf8): https://commits.webkit.org/269631@main

Reviewed commits have been landed. Closing PR #19393 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit bfc8bf8 into WebKit:main Oct 21, 2023
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Oct 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebKit Misc. For miscellaneous bugs in the WebKit framework (and not JavaScriptCore or WebCore).
Projects
None yet
4 participants