Revert "Fix enrollment cancelation race conditions." (f4dc031e) · Commits · e / os / android_frameworks_base

core/java/android/hardware/face/FaceManager.java

+22 −27

Original line number	Diff line number	Diff line
		@@ -306,21 +306,22 @@ public class FaceManager implements BiometricAuthenticator, BiometricFaceConstan
		throw new IllegalArgumentException("Must supply an enrollment callback");
		}

		if (cancel != null && cancel.isCanceled()) {
		if (cancel != null) {
		if (cancel.isCanceled()) {
		Slog.w(TAG, "enrollment already canceled");
		return;
		} else {
		cancel.setOnCancelListener(new OnEnrollCancelListener());
		}
		}

		if (mService != null) {
		try {
		mEnrollmentCallback = callback;
		Trace.beginSection("FaceManager#enroll");
		final long enrollId = mService.enroll(userId, mToken, hardwareAuthToken,
		mServiceReceiver, mContext.getOpPackageName(), disabledFeatures,
		previewSurface, debugConsent);
		if (cancel != null) {
		cancel.setOnCancelListener(new OnEnrollCancelListener(enrollId));
		}
		mService.enroll(userId, mToken, hardwareAuthToken, mServiceReceiver,
		mContext.getOpPackageName(), disabledFeatures, previewSurface,
		debugConsent);
		} catch (RemoteException e) {
		Slog.w(TAG, "Remote exception in enroll: ", e);
		// Though this may not be a hardware issue, it will cause apps to give up or
		@@ -358,20 +359,21 @@ public class FaceManager implements BiometricAuthenticator, BiometricFaceConstan
		throw new IllegalArgumentException("Must supply an enrollment callback");
		}

		if (cancel != null && cancel.isCanceled()) {
		if (cancel != null) {
		if (cancel.isCanceled()) {
		Slog.w(TAG, "enrollRemotely is already canceled.");
		return;
		} else {
		cancel.setOnCancelListener(new OnEnrollCancelListener());
		}
		}

		if (mService != null) {
		try {
		mEnrollmentCallback = callback;
		Trace.beginSection("FaceManager#enrollRemotely");
		final long enrolId = mService.enrollRemotely(userId, mToken, hardwareAuthToken,
		mServiceReceiver, mContext.getOpPackageName(), disabledFeatures);
		if (cancel != null) {
		cancel.setOnCancelListener(new OnEnrollCancelListener(enrolId));
		}
		mService.enrollRemotely(userId, mToken, hardwareAuthToken, mServiceReceiver,
		mContext.getOpPackageName(), disabledFeatures);
		} catch (RemoteException e) {
		Slog.w(TAG, "Remote exception in enrollRemotely: ", e);
		// Though this may not be a hardware issue, it will cause apps to give up or
		@@ -711,10 +713,10 @@ public class FaceManager implements BiometricAuthenticator, BiometricFaceConstan
		}
		}

		private void cancelEnrollment(long requestId) {
		private void cancelEnrollment() {
		if (mService != null) {
		try {
		mService.cancelEnrollment(mToken, requestId);
		mService.cancelEnrollment(mToken);
		} catch (RemoteException e) {
		throw e.rethrowFromSystemServer();
		}
		@@ -1098,16 +1100,9 @@ public class FaceManager implements BiometricAuthenticator, BiometricFaceConstan
		}

		private class OnEnrollCancelListener implements OnCancelListener {
		private final long mAuthRequestId;

		private OnEnrollCancelListener(long id) {
		mAuthRequestId = id;
		}

		@Override
		public void onCancel() {
		Slog.d(TAG, "Cancel face enrollment requested for: " + mAuthRequestId);
		cancelEnrollment(mAuthRequestId);
		cancelEnrollment();
		}
		}

core/java/android/hardware/face/IFaceService.aidl

+4 −5

Original line number	Diff line number	Diff line
		@@ -76,16 +76,15 @@ interface IFaceService {
		void cancelAuthenticationFromService(int sensorId, IBinder token, String opPackageName, long requestId);

		// Start face enrollment
		long enroll(int userId, IBinder token, in byte [] hardwareAuthToken, IFaceServiceReceiver receiver,
		String opPackageName, in int [] disabledFeatures,
		in Surface previewSurface, boolean debugConsent);
		void enroll(int userId, IBinder token, in byte [] hardwareAuthToken, IFaceServiceReceiver receiver,
		String opPackageName, in int [] disabledFeatures, in Surface previewSurface, boolean debugConsent);

		// Start remote face enrollment
		long enrollRemotely(int userId, IBinder token, in byte [] hardwareAuthToken, IFaceServiceReceiver receiver,
		void enrollRemotely(int userId, IBinder token, in byte [] hardwareAuthToken, IFaceServiceReceiver receiver,
		String opPackageName, in int [] disabledFeatures);

		// Cancel enrollment in progress
		void cancelEnrollment(IBinder token, long requestId);
		void cancelEnrollment(IBinder token);

		// Removes the specified face enrollment for the specified userId.
		void remove(IBinder token, int faceId, int userId, IFaceServiceReceiver receiver,

core/java/android/hardware/fingerprint/FingerprintManager.java

+12 −18

Original line number	Diff line number	Diff line
		@@ -183,16 +183,9 @@ public class FingerprintManager implements BiometricAuthenticator, BiometricFing
		}

		private class OnEnrollCancelListener implements OnCancelListener {
		private final long mAuthRequestId;

		private OnEnrollCancelListener(long id) {
		mAuthRequestId = id;
		}

		@Override
		public void onCancel() {
		Slog.d(TAG, "Cancel fingerprint enrollment requested for: " + mAuthRequestId);
		cancelEnrollment(mAuthRequestId);
		cancelEnrollment();
		}
		}

		@@ -653,19 +646,20 @@ public class FingerprintManager implements BiometricAuthenticator, BiometricFing
		throw new IllegalArgumentException("Must supply an enrollment callback");
		}

		if (cancel != null && cancel.isCanceled()) {
		if (cancel != null) {
		if (cancel.isCanceled()) {
		Slog.w(TAG, "enrollment already canceled");
		return;
		} else {
		cancel.setOnCancelListener(new OnEnrollCancelListener());
		}
		}

		if (mService != null) {
		try {
		mEnrollmentCallback = callback;
		final long enrollId = mService.enroll(mToken, hardwareAuthToken, userId,
		mServiceReceiver, mContext.getOpPackageName(), enrollReason);
		if (cancel != null) {
		cancel.setOnCancelListener(new OnEnrollCancelListener(enrollId));
		}
		mService.enroll(mToken, hardwareAuthToken, userId, mServiceReceiver,
		mContext.getOpPackageName(), enrollReason);
		} catch (RemoteException e) {
		Slog.w(TAG, "Remote exception in enroll: ", e);
		// Though this may not be a hardware issue, it will cause apps to give up or try
		@@ -1308,9 +1302,9 @@ public class FingerprintManager implements BiometricAuthenticator, BiometricFing
		return allSensors.isEmpty() ? null : allSensors.get(0);
		}

		private void cancelEnrollment(long requestId) {
		private void cancelEnrollment() {
		if (mService != null) try {
		mService.cancelEnrollment(mToken, requestId);
		mService.cancelEnrollment(mToken);
		} catch (RemoteException e) {
		throw e.rethrowFromSystemServer();
		}

core/java/android/hardware/fingerprint/IFingerprintService.aidl

+2 −2

Original line number	Diff line number	Diff line
		@@ -84,11 +84,11 @@ interface IFingerprintService {
		void cancelAuthenticationFromService(int sensorId, IBinder token, String opPackageName, long requestId);

		// Start fingerprint enrollment
		long enroll(IBinder token, in byte [] hardwareAuthToken, int userId, IFingerprintServiceReceiver receiver,
		void enroll(IBinder token, in byte [] hardwareAuthToken, int userId, IFingerprintServiceReceiver receiver,
		String opPackageName, int enrollReason);

		// Cancel enrollment in progress
		void cancelEnrollment(IBinder token, long requestId);
		void cancelEnrollment(IBinder token);

		// Any errors resulting from this call will be returned to the listener
		void remove(IBinder token, int fingerId, int userId, IFingerprintServiceReceiver receiver,

services/core/java/com/android/server/biometrics/sensors/BaseClientMonitor.java

+2 −4

Original line number	Diff line number	Diff line
		@@ -16,8 +16,6 @@

		package com.android.server.biometrics.sensors;

		import static com.android.internal.annotations.VisibleForTesting.Visibility;

		import android.annotation.NonNull;
		import android.annotation.Nullable;
		import android.content.Context;
		@@ -50,6 +48,7 @@ public abstract class BaseClientMonitor extends LoggableMonitor
		* Interface that ClientMonitor holders should use to receive callbacks.
		*/
		public interface Callback {

		/**
		* Invoked when the ClientMonitor operation has been started (e.g. reached the head of
		* the queue and becomes the current operation).
		@@ -204,8 +203,7 @@ public abstract class BaseClientMonitor extends LoggableMonitor
		}

		/** Signals this operation has completed its lifecycle and should no longer be used. */
		@VisibleForTesting(visibility = Visibility.PACKAGE)
		public void destroy() {
		void destroy() {
		mAlreadyDone = true;
		if (mToken != null) {
		try {