Merge "Revert "SharedMemory: break Cleaner reference cycle." Bug: 138422309" into qt-dev (868ac4f3) · Commits · e / os / android_frameworks_base

core/java/android/os/SharedMemory.java

+4 −6

Original line number	Diff line number	Diff line
		@@ -62,7 +62,7 @@ public final class SharedMemory implements Parcelable, Closeable {

		mMemoryRegistration = new MemoryRegistration(mSize);
		mCleaner = Cleaner.create(mFileDescriptor,
		new Closer(mFileDescriptor.getInt$(), mMemoryRegistration));
		new Closer(mFileDescriptor, mMemoryRegistration));
		}

		/**
		@@ -290,10 +290,10 @@ public final class SharedMemory implements Parcelable, Closeable {
		* Cleaner that closes the FD
		*/
		private static final class Closer implements Runnable {
		private int mFd;
		private FileDescriptor mFd;
		private MemoryRegistration mMemoryReference;

		private Closer(int fd, MemoryRegistration memoryReference) {
		private Closer(FileDescriptor fd, MemoryRegistration memoryReference) {
		mFd = fd;
		mMemoryReference = memoryReference;
		}
		@@ -301,9 +301,7 @@ public final class SharedMemory implements Parcelable, Closeable {
		@Override
		public void run() {
		try {
		FileDescriptor fd = new FileDescriptor();
		fd.setInt$(mFd);
		Os.close(fd);
		Os.close(mFd);
		} catch (ErrnoException e) { /* swallow error */ }
		mMemoryReference.release();
		mMemoryReference = null;