finally block ensures release() is called even if exceptions occur

amethyst/src/main/java/com/vitorpamplona/amethyst/service/uploads/VideoCompressionHelper.kt

@@ -338,9 +338,10 @@ class VideoCompressionHelper {
     private fun getVideoInfo(
         uri: Uri,
         context: Context,
-    ): VideoInfo? =
+    ): VideoInfo? {
-        try {
+        var retriever: MediaMetadataRetriever? = null
-            val retriever = MediaMetadataRetriever()
+        return try {
+            retriever = MediaMetadataRetriever()
             retriever.setDataSource(context, uri)
             val width = retriever.prepareVideoWidth()
             val height = retriever.prepareVideoHeight()
@@ -350,8 +351,6 @@ class VideoCompressionHelper {
             val framerateString = retriever.extractMetadata(MediaMetadataRetriever.METADATA_KEY_CAPTURE_FRAMERATE)
             val framerate = framerateString?.toFloatOrNull() ?: 30.0f
 
-            retriever.release()
-
             if (width != null && height != null && width > 0 && height > 0) {
                 // Account for rotation
                 val resolution =
@@ -367,5 +366,12 @@ class VideoCompressionHelper {
         } catch (e: Exception) {
             Log.w("VideoCompressionHelper", "Failed to get video resolution: ${e.message}")
             null
+        } finally {
+            try {
+                retriever?.release()
+            } catch (e: Exception) {
+                Log.w("VideoCompressionHelper", "Failed to release MediaMetadataRetriever: ${e.message}")
+            }
         }
+    }
 }