[webkit-changes] [WebKit/WebKit] c6a8bc: Use-after-free in RTCPeerConnection::processIceTra...

youennf noreply at github.com
Wed Dec 20 16:47:56 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c6a8bc14421a84ab750428015a97cac14185e36a
      https://github.com/WebKit/WebKit/commit/c6a8bc14421a84ab750428015a97cac14185e36a
  Author: Youenn Fablet <youennf at gmail.com>
  Date:   2023-12-20 (Wed, 20 Dec 2023)

  Changed paths:
    M LayoutTests/platform/glib/TestExpectations
    A LayoutTests/webrtc/processIceTransportStateChange-gc-expected.txt
    A LayoutTests/webrtc/processIceTransportStateChange-gc.html
    M Source/WebCore/Modules/mediastream/RTCIceTransport.cpp
    M Source/WebCore/Modules/mediastream/RTCIceTransport.h

  Log Message:
  -----------
  Use-after-free in RTCPeerConnection::processIceTransportStateChange
rdar://117526483

Reviewed by Jean-Yves Avenard.

RTCIceTransport is calling RTCPeerConnection::processIceTransportStateChange without protecting its RTCPeerConnection.
processIceTransportStateChange can trigger JS execution so we need to protect the RTCPeerConnection.
Make RTCIceTransport do so, and update RTCIceTransport connection getter to return a RefPtr instead of a raw pointer.

* LayoutTests/webrtc/processIceTransportStateChange-gc-expected.txt: Added.
* LayoutTests/webrtc/processIceTransportStateChange-gc.html: Added.
* Source/WebCore/Modules/mediastream/RTCDtlsTransport.cpp:
(WebCore::RTCDtlsTransport::onStateChanged):
* Source/WebCore/Modules/mediastream/RTCIceTransport.cpp:
(WebCore::RTCIceTransport::onStateChanged):
* Source/WebCore/Modules/mediastream/RTCIceTransport.h:
(WebCore::RTCIceTransport::connection const):

Originally-landed-as: 267815.446 at safari-7617-branch (8be2b8b167a1). rdar://119595786
Canonical link: https://commits.webkit.org/272392@main




More information about the webkit-changes mailing list