| Summary: | GPU Process: Sandbox violations under IOSurface::maximumSize in the Web Content process | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Tim Horton <thorton> | ||||||
| Component: | New Bugs | Assignee: | Tim Horton <thorton> | ||||||
| Status: | RESOLVED FIXED | ||||||||
| Severity: | Normal | CC: | pvollan, rniwa, sabouhallawa, simon.fraser, webkit-bug-importer, wenson_hsieh | ||||||
| Priority: | P2 | Keywords: | InRadar | ||||||
| Version: | WebKit Nightly Build | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=219492 https://bugs.webkit.org/show_bug.cgi?id=235526 https://bugs.webkit.org/show_bug.cgi?id=221346 |
||||||||
| Attachments: |
|
||||||||
|
Description
Tim Horton
2020-12-03 00:59:32 PST
Created attachment 415282 [details]
Patch
Comment on attachment 415282 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415282&action=review > Source/WebCore/platform/graphics/cocoa/IOSurface.mm:248 > + static WTF::Optional<IntSize> maximumSize; > + return maximumSize; Can we assert that this is only called in the main thread? Created attachment 415287 [details]
Patch
Committed r270392: <https://trac.webkit.org/changeset/270392> All reviewed patches have been landed. Closing bug and clearing flags on attachment 415287 [details]. Comment on attachment 415287 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=415287&action=review > Source/WebKit/ChangeLog:17 > + Fetch the maximum allowed size of an IOSurface on the current hardware It will never change when switching between integrated and discrete? I hope not, since we don’t recreate all existing surfaces when that happens. But it’s a good point. |