fix(jni): delete local ref

alibaba · Jun 11, 2020 · 01cc6e0 · 01cc6e0
1 parent bd2ca4b
commit 01cc6e0
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/platform/Android/source/premierlibrary/src/main/jni/player/JavaPlayerConfig.cpp b/platform/Android/source/premierlibrary/src/main/jni/player/JavaPlayerConfig.cpp
@@ -206,15 +206,18 @@ MediaPlayerConfig JavaPlayerConfig::convertTo(JNIEnv *env, jobject playerConfig)
             for (int i = 0; i < len; i++) {
                 jstring           header     = static_cast<jstring>(env->GetObjectArrayElement(
                                                    headersArray, i));
-                GetStringUTFChars tmpHeader(env, header);
-                char              *ch_header = tmpHeader.getChars();
+                {
+                    GetStringUTFChars tmpHeader(env, header);
+                    char              *ch_header = tmpHeader.getChars();
 
-                if (ch_header != nullptr) {
-                    config.customHeaders.push_back(std::string(ch_header));
+                    if (ch_header != nullptr) {
+                        config.customHeaders.push_back(std::string(ch_header));
+                    }
                 }
+                env->DeleteLocalRef(header);
             }
         }
     }
 
     return config;
-}
+}