Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0b0856901df8b016acfa66a7677c7552d7b0f1bb
      
https://github.com/WebKit/WebKit/commit/0b0856901df8b016acfa66a7677c7552d7b0f1bb
  Author: Abrar Rahman Protyasha <a_protya...@apple.com>
  Date:   2024-05-25 (Sat, 25 May 2024)

  Changed paths:
    M Source/WebKit/UIProcess/ViewGestureController.cpp

  Log Message:
  -----------
  UI process crash due to null pointer dereference under 
ViewGestureController::applyMagnification()
https://bugs.webkit.org/show_bug.cgi?id=274710
rdar://128521737

Reviewed by Wenson Hsieh.

We have received a few crash reports with the following signature:

```
Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       KERN_INVALID_ADDRESS at 0x0000000000000000
Exception Codes:       0x0000000000000001, 0x0000000000000000

Thread 0 Crashed::   Dispatch queue: com.apple.main-thread
0   com.apple.WebKit                       0x1ada630ec 
WebKit::ViewGestureController::applyMagnification() + 208
1   com.apple.WebKit                       0x1ad8dee50 
WebKit::ViewGestureController::handleMagnificationGestureEvent(NSEvent*, 
WebCore::FloatPoint) + 296
2   com.apple.WebKit                       0x1ad9160e4 
WebKit::WebViewImpl::magnifyWithEvent(NSEvent*) + 164
```

... which is simply a null pointer dereference in applyMagnification().
The only pointer dereference in that function occurs on the
DrawingAreaProxy, so let's speculatively guard said dereference.

* Source/WebKit/UIProcess/ViewGestureController.cpp:
(WebKit::ViewGestureController::applyMagnification):

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to