[FFmpeg-devel,3/4] avcodec/mediacodec_wrapper: fix a potential local reference leak in ff_AMediaCodec_getCodecNameByType()

Submitted by Matthieu Bouron on July 4, 2019, 1:43 p.m.

Details

Message ID 20190704134348.6071-3-matthieu.bouron@gmail.com
State Accepted
Commit 3f232d713db3cb665c265387c7009904c2e85e58
Headers show

Commit Message

Matthieu Bouron July 4, 2019, 1:43 p.m.
---
 libavcodec/mediacodec_wrapper.c | 5 +++++
 1 file changed, 5 insertions(+)

Patch hide | download patch | download mbox

diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c
index e2df07cb41..70e1e7cae1 100644
--- a/libavcodec/mediacodec_wrapper.c
+++ b/libavcodec/mediacodec_wrapper.c
@@ -469,6 +469,11 @@  char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
                     goto done;
                 }
 
+                if (codec_name) {
+                    (*env)->DeleteLocalRef(env, codec_name);
+                    codec_name = NULL;
+                }
+
                 /* Skip software decoders */
                 if (
                     strstr(name, "OMX.google") ||