Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit db92babb authored by Zimuzo Ezeozue's avatar Zimuzo Ezeozue Committed by Automerger Merge Worker
Browse files

Merge "Fix fd leak while bypassing transcoding in media APIs" into sc-dev am: 3fefc5c3 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/15423590

Change-Id: Idc2f4c83f95723970d9f9c7eb0b1851e8247fff7
parents 9f00bfc9 3fefc5c3

core/java/android/os/FileUtils.java

+9 −9
Original line number Diff line number Diff line
@@ -1460,15 +1460,15 @@ public final class FileUtils { 
    /** {@hide} */

    @VisibleForTesting

    public static ParcelFileDescriptor convertToModernFd(FileDescriptor fd) {

        try {

        Context context = AppGlobals.getInitialApplication();

        if (UserHandle.getAppId(Process.myUid()) == getMediaProviderAppId(context)) {

            // Never convert modern fd for MediaProvider, because this requires

            // MediaStore#scanFile and can cause infinite loops when MediaProvider scans

            return null;

        }

            return MediaStore.getOriginalMediaFormatFileDescriptor(context,

                    ParcelFileDescriptor.dup(fd));



        try (ParcelFileDescriptor dupFd = ParcelFileDescriptor.dup(fd)) {

            return MediaStore.getOriginalMediaFormatFileDescriptor(context, dupFd);

        } catch (Exception e) {

            Log.d(TAG, "Failed to convert to modern format file descriptor", e);

            return null;

media/java/android/media/ExifInterface.java

+8 −1
Original line number Diff line number Diff line
@@ -1573,6 +1573,9 @@ public class ExifInterface { 
            if (isFdDuped) {

                closeFileDescriptor(fileDescriptor);

            }

            if (modernFd != null) {

                modernFd.close();

            }

        }

    }
@@ -2554,12 +2557,13 @@ public class ExifInterface { 


    private void initForFilename(String filename) throws IOException {

        FileInputStream in = null;

        ParcelFileDescriptor modernFd = null;

        mAssetInputStream = null;

        mFilename = filename;

        mIsInputStream = false;

        try {

            in = new FileInputStream(filename);

            ParcelFileDescriptor modernFd = FileUtils.convertToModernFd(in.getFD());

            modernFd = FileUtils.convertToModernFd(in.getFD());

            if (modernFd != null) {

                closeQuietly(in);

                in = new FileInputStream(modernFd.getFileDescriptor());
@@ -2570,6 +2574,9 @@ public class ExifInterface { 
            loadAttributes(in);

        } finally {

            closeQuietly(in);

            if (modernFd != null) {

                modernFd.close();

            }

        }

    }

media/java/android/media/MediaMetadataRetriever.java

+12 −5
Original line number Diff line number Diff line
@@ -36,6 +36,7 @@ import android.os.IBinder; 
import android.os.ParcelFileDescriptor;

import android.os.SystemProperties;

import android.text.TextUtils;

import android.util.Log;



import java.io.FileDescriptor;

import java.io.FileInputStream;
@@ -52,6 +53,8 @@ import java.util.Map; 
 * frame and meta data from an input media file.

 */

public class MediaMetadataRetriever implements AutoCloseable {

    private static final String TAG = "MediaMetadataRetriever";



    // borrowed from ExoPlayer

    private static final String[] STANDARD_GENRES = new String[] {

            // These are the official ID3v1 genres.
@@ -301,12 +304,16 @@ public class MediaMetadataRetriever implements AutoCloseable { 
     */

    public void setDataSource(FileDescriptor fd, long offset, long length)

            throws IllegalArgumentException  {

        ParcelFileDescriptor modernFd = FileUtils.convertToModernFd(fd);



        try (ParcelFileDescriptor modernFd = FileUtils.convertToModernFd(fd)) {

            if (modernFd == null) {

                _setDataSource(fd, offset, length);

            } else {

                _setDataSource(modernFd.getFileDescriptor(), offset, length);

            }

        } catch (IOException e) {

            Log.w(TAG, "Ignoring IO error while setting data source", e);

        }

    }



    private native void _setDataSource(FileDescriptor fd, long offset, long length)

media/java/android/media/MediaPlayer.java

+8 −5
Original line number Diff line number Diff line
@@ -1271,12 +1271,15 @@ public class MediaPlayer extends PlayerBase 
     */

    public void setDataSource(FileDescriptor fd, long offset, long length)

            throws IOException, IllegalArgumentException, IllegalStateException {

        ParcelFileDescriptor modernFd = FileUtils.convertToModernFd(fd);

        try (ParcelFileDescriptor modernFd = FileUtils.convertToModernFd(fd)) {

            if (modernFd == null) {

                _setDataSource(fd, offset, length);

            } else {

                _setDataSource(modernFd.getFileDescriptor(), offset, length);

            }

        } catch (IOException e) {

            Log.w(TAG, "Ignoring IO error while setting data source", e);

        }

    }



    private native void _setDataSource(FileDescriptor fd, long offset, long length)