From 4aaee3844e8b829ada611496ebca1ae80495b0c0 Mon Sep 17 00:00:00 2001 From: Greg Kaiser Date: Tue, 7 Jan 2020 09:42:10 -0800 Subject: android_media_tv_Tuner: Initial member We initial the 'mDvrMQEventFlag' member in the constructor, so that if we end up invoking the destructor with having called android_media_tv_Tuner_configure_dvr() first, we won't use uninitialized memory as a pointer. Test: TreeHugger Change-Id: Iea3efd45c5d68a90ec36ebf95f00cb1e0a0b054d --- media/jni/android_media_tv_Tuner.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/media/jni/android_media_tv_Tuner.cpp b/media/jni/android_media_tv_Tuner.cpp index 3562782eb275..c88e2d20ca67 100644 --- a/media/jni/android_media_tv_Tuner.cpp +++ b/media/jni/android_media_tv_Tuner.cpp @@ -100,7 +100,7 @@ void DvrCallback::setDvr(const jobject dvr) { /////////////// Dvr /////////////////////// -Dvr::Dvr(sp sp, jweak obj) : mDvrSp(sp), mDvrObj(obj) {} +Dvr::Dvr(sp sp, jweak obj) : mDvrSp(sp), mDvrObj(obj), mDvrMQEventFlag(nullptr) {} Dvr::~Dvr() { EventFlag::deleteEventFlag(&mDvrMQEventFlag); -- cgit v1.2.3-59-g8ed1b