From b463ae1ac7c971687dc8cf59b7e45baa5a33156d Mon Sep 17 00:00:00 2001 From: Byoungchan Lee Date: Wed, 5 Jan 2022 00:11:41 +0900 Subject: [PATCH] Add a sanity check of RTCVideoFrame in RTCMTLVideoView Bug: webrtc:13490 Change-Id: I6cca1966c660b2fa36d2d77a816cdd734cce7cf2 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/244420 Reviewed-by: Henrik Andreassson Reviewed-by: Peter Hanspers Commit-Queue: Henrik Andreassson Cr-Commit-Position: refs/heads/main@{#35636} --- sdk/objc/components/renderer/metal/RTCMTLVideoView.m | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sdk/objc/components/renderer/metal/RTCMTLVideoView.m b/sdk/objc/components/renderer/metal/RTCMTLVideoView.m index 4c50bcf9c1..c5d9e4385f 100644 --- a/sdk/objc/components/renderer/metal/RTCMTLVideoView.m +++ b/sdk/objc/components/renderer/metal/RTCMTLVideoView.m @@ -139,7 +139,8 @@ NSAssert(view == self.metalView, @"Receiving draw callbacks from foreign instance."); RTC_OBJC_TYPE(RTCVideoFrame) *videoFrame = self.videoFrame; // Skip rendering if we've already rendered this frame. - if (!videoFrame || videoFrame.timeStampNs == self.lastFrameTimeNs) { + if (!videoFrame || videoFrame.width <= 0 || videoFrame.height <= 0 || + videoFrame.timeStampNs == self.lastFrameTimeNs) { return; }