| commit | 9c41a9f1ac56d795312e8a0b2670302c47a97aa8 | [log] [tgz] |
|---|---|---|
| author | Harish Mahendrakar <harish.mahendrakar@ittiam.com> | Fri Jan 03 16:33:25 2020 -0800 |
| committer | Harish Mahendrakar <harish.mahendrakar@ittiam.com> | Wed Jan 08 23:48:41 2020 +0000 |
| tree | bb55ceebe89a857cb2f7764953cae1717a0f5d5c | |
| parent | e5f01a7d5edd517c3a52ca11e6908aead47ba916 [diff] |
benchmark: Fix NPD in Encode::deInitCodec() In some cases where initCodec() fails, deInitCodec() should not call AMediaCodec_stop() as mCodec is NULL. Bug: 147158050 Test: poc in bug Change-Id: I5dc1bf1fed8fe086e16a9f07e1289980333b8a12
diff --git a/media/tests/benchmark/src/native/encoder/Encoder.cpp b/media/tests/benchmark/src/native/encoder/Encoder.cpp index f119cf3..8dc7cc6 100644 --- a/media/tests/benchmark/src/native/encoder/Encoder.cpp +++ b/media/tests/benchmark/src/native/encoder/Encoder.cpp
@@ -159,6 +159,7 @@ AMediaFormat_delete(mFormat); mFormat = nullptr; } + if (!mCodec) return; AMediaCodec_stop(mCodec); AMediaCodec_delete(mCodec); int64_t eTime = mStats->getCurTime();