commit 3e1484e2807f8c19cc26d722a91226c62361786f
author Michael Froman <mfroman@mozilla.com> Wed Aug 16 21:31:19 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Sep 18 15:15:34 2023
tree 9b823605bfbe93ef4551bdb30b2d6ee205c09c98
parent 96bc094d38e1d4d18f897d39a609897b9fdcd857

Check ConvertToI420 result for all errors in VideoCaptureImpl::IncomingFrame

Bug: webrtc:15415
Change-Id: Ia303e1803d8238c4db68c7dc8d207b0ccfccadba
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/316343
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Commit-Queue: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#40762}

modules/video_capture/video_capture_impl.cc

diff --git a/modules/video_capture/video_capture_impl.cc b/modules/video_capture/video_capture_impl.cc
index a1461fb..428253b 100644
--- a/modules/video_capture/video_capture_impl.cc
+++ b/modules/video_capture/video_capture_impl.cc
@@ -214,7 +214,7 @@
       buffer.get()->StrideV(), 0, 0,  // No Cropping
       width, height, target_width, target_height, rotation_mode,
       ConvertVideoType(frameInfo.videoType));
-  if (conversionResult < 0) {
+  if (conversionResult != 0) {
     RTC_LOG(LS_ERROR) << "Failed to convert capture frame from type "
                       << static_cast<int>(frameInfo.videoType) << "to I420.";
     return -1;