Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c31e0dd75e709ee3ffec35dc0ece60754149fdc8
      
https://github.com/WebKit/WebKit/commit/c31e0dd75e709ee3ffec35dc0ece60754149fdc8
  Author: Alex Christensen <achristen...@apple.com>
  Date:   2023-11-17 (Fri, 17 Nov 2023)

  Changed paths:
    M Source/WebKit/CMakeLists.txt
    M Source/WebKit/DerivedSources-input.xcfilelist
    M Source/WebKit/DerivedSources.make
    M Source/WebKit/Shared/Cocoa/SandboxExtensionCocoa.mm
    M Source/WebKit/Shared/SandboxExtension.h
    A Source/WebKit/Shared/SandboxExtension.serialization.in
    M Source/WebKit/WebKit.xcodeproj/project.pbxproj

  Log Message:
  -----------
  Generate serialization of SandboxExtensionHandle
https://bugs.webkit.org/show_bug.cgi?id=264994
rdar://118539219

Reviewed by Chris Dumez.

All the transformations to get this to work are pretty straightforward,
except the if (dataReference.empty()) check to deserialize a valid, null
SandboxExtensionHandle if sandboxExtensionForType somehow returns an empty
string is replaced by a fast strlen-is-greater-than-0 check in
SandboxExtensionImpl::create to return an empty handle in the creation
process rather than failing to decode and killing the creation process.
This matches existing behavior as close as possible.

* Source/WebKit/CMakeLists.txt:
* Source/WebKit/DerivedSources-input.xcfilelist:
* Source/WebKit/DerivedSources.make:
* Source/WebKit/Shared/Cocoa/SandboxExtensionCocoa.mm:
(WebKit::SandboxExtensionImpl::create):
(WebKit::SandboxExtensionImpl::SandboxExtensionImpl):
(WebKit::SandboxExtensionImpl::~SandboxExtensionImpl):
(WebKit::SandboxExtensionImpl::consume):
(WebKit::SandboxExtensionImpl::invalidate):
(WebKit::SandboxExtensionImpl::getSerializedFormat):
(WebKit::SandboxExtensionImpl::sandboxExtensionForType):
(): Deleted.
(WebKit::SandboxExtensionHandle::encode): Deleted.
(WebKit::SandboxExtensionHandle::decode): Deleted.
* Source/WebKit/Shared/SandboxExtension.h:
(WebKit::SandboxExtensionImpl::SandboxExtensionImpl):
(WebKit::SandboxExtensionHandle::SandboxExtensionHandle):
(WebKit::SandboxExtensionHandle::takeImpl):
(WebKit::SandboxExtensionHandle::~SandboxExtensionHandle):
(WebKit::SandboxExtensionHandle::encode): Deleted.
(WebKit::SandboxExtensionHandle::decode): Deleted.
* Source/WebKit/Shared/SandboxExtension.serialization.in: Added.
* Source/WebKit/WebKit.xcodeproj/project.pbxproj:

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


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to