close inputstream and datasource to prevent leaks
This commit is contained in:
parent
c3778e36c6
commit
a5292aa669
|
@ -740,18 +740,18 @@ class PhotoFragment : ViewPagerFragment() {
|
|||
}
|
||||
|
||||
private fun checkIfPanorama() {
|
||||
mIsPanorama = try {
|
||||
val inputStream = if (mMedium.path.startsWith("content:/")) {
|
||||
mIsPanorama = try {
|
||||
if (mMedium.path.startsWith("content:/")) {
|
||||
requireContext().contentResolver.openInputStream(Uri.parse(mMedium.path))
|
||||
} else {
|
||||
File(mMedium.path).inputStream()
|
||||
}.use {
|
||||
val imageParser = JpegImageParser().getXmpXml(ByteSourceInputStream(it, mMedium.name), HashMap<String, Any>())
|
||||
imageParser.contains("GPano:UsePanoramaViewer=\"True\"", true) ||
|
||||
imageParser.contains("<GPano:UsePanoramaViewer>True</GPano:UsePanoramaViewer>", true) ||
|
||||
imageParser.contains("GPano:FullPanoWidthPixels=") ||
|
||||
imageParser.contains("GPano:ProjectionType>Equirectangular")
|
||||
}
|
||||
|
||||
val imageParser = JpegImageParser().getXmpXml(ByteSourceInputStream(inputStream, mMedium.name), HashMap<String, Any>())
|
||||
imageParser.contains("GPano:UsePanoramaViewer=\"True\"", true) ||
|
||||
imageParser.contains("<GPano:UsePanoramaViewer>True</GPano:UsePanoramaViewer>", true) ||
|
||||
imageParser.contains("GPano:FullPanoWidthPixels=") ||
|
||||
imageParser.contains("GPano:ProjectionType>Equirectangular")
|
||||
} catch (e: Exception) {
|
||||
false
|
||||
} catch (e: OutOfMemoryError) {
|
||||
|
|
|
@ -362,6 +362,7 @@ import java.io.FileInputStream
|
|||
try {
|
||||
fileDataSource.open(dataSpec)
|
||||
} catch (e: Exception) {
|
||||
fileDataSource.close()
|
||||
activity?.showErrorToast(e)
|
||||
return
|
||||
}
|
||||
|
@ -370,6 +371,8 @@ import java.io.FileInputStream
|
|||
val mediaSource: MediaSource = ProgressiveMediaSource.Factory(factory)
|
||||
.createMediaSource(MediaItem.fromUri(fileDataSource.uri!!))
|
||||
|
||||
fileDataSource.close()
|
||||
|
||||
mPlayOnPrepared = true
|
||||
|
||||
mExoPlayer = ExoPlayer.Builder(requireContext())
|
||||
|
|
Loading…
Reference in New Issue