From 9a4c93b7d73b6f6ee755f309744f6ec35664cac7 Mon Sep 17 00:00:00 2001
From: Elad Alon <eladalon@webrtc.org>
Date: Thu, 16 May 2019 14:30:57 +0200
Subject: [PATCH] Add DCHECK in LibvpxVp8Encoder

Ensure that frame_buffer_controller_ does not get assigned null
by the factory.

Bug: None
Change-Id: I84e141ae0390cd024863f88cdcdc79b8b13e7c64
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/137043
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Commit-Queue: Elad Alon <eladalon@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27962}
---
 modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc b/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc
index 0bb08c7e8a..c29832d277 100644
--- a/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc
+++ b/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc
@@ -490,6 +490,7 @@ int LibvpxVp8Encoder::InitEncode(const VideoCodec* inst,
     Vp8TemporalLayersFactory factory;
     frame_buffer_controller_ = factory.Create(*inst);
   }
+  RTC_DCHECK(frame_buffer_controller_);
 
   number_of_cores_ = number_of_cores;
   timestamp_ = 0;