Merge "Fix onClientFinished for generate and revoke challenge" into sc-dev

This commit is contained in:
TreeHugger Robot
2021-04-30 05:49:48 +00:00
committed by Android (Google) Code Review
9 changed files with 12 additions and 5 deletions

View File

@@ -42,7 +42,6 @@ public abstract class RevokeChallengeClient<T> extends HalClientMonitor<T> {
super.start(callback);
startHalOperation();
mCallback.onClientFinished(this, true /* success */);
}
@Override

View File

@@ -45,6 +45,7 @@ public class FaceGenerateChallengeClient extends GenerateChallengeClient<ISessio
getFreshDaemon().generateChallenge();
} catch (RemoteException e) {
Slog.e(TAG, "Unable to generateChallenge", e);
mCallback.onClientFinished(this, false /* success */);
}
}

View File

@@ -48,6 +48,7 @@ public class FaceRevokeChallengeClient extends RevokeChallengeClient<ISession> {
getFreshDaemon().revokeChallenge(mChallenge);
} catch (RemoteException e) {
Slog.e(TAG, "Unable to revokeChallenge", e);
mCallback.onClientFinished(this, false /* success */);
}
}

View File

@@ -67,6 +67,7 @@ public class FaceGenerateChallengeClient extends GenerateChallengeClient<IBiomet
}
} catch (RemoteException e) {
Slog.e(TAG, "generateChallenge failed", e);
mCallback.onClientFinished(this, false /* success */);
}
}
}

View File

@@ -43,8 +43,10 @@ public class FaceRevokeChallengeClient extends RevokeChallengeClient<IBiometrics
protected void startHalOperation() {
try {
getFreshDaemon().revokeChallenge();
mCallback.onClientFinished(this, true /* success */);
} catch (RemoteException e) {
Slog.e(TAG, "revokeChallenge failed", e);
mCallback.onClientFinished(this, false /* success */);
}
}
}

View File

@@ -47,18 +47,17 @@ class FingerprintGenerateChallengeClient extends GenerateChallengeClient<ISessio
getFreshDaemon().generateChallenge();
} catch (RemoteException e) {
Slog.e(TAG, "Unable to generateChallenge", e);
mCallback.onClientFinished(this, false /* success */);
}
}
void onChallengeGenerated(int sensorId, int userId, long challenge) {
try {
getListener().onChallengeGenerated(sensorId, challenge);
mCallback.onClientFinished(FingerprintGenerateChallengeClient.this,
true /* success */);
mCallback.onClientFinished(this, true /* success */);
} catch (RemoteException e) {
Slog.e(TAG, "Unable to send challenge", e);
mCallback.onClientFinished(FingerprintGenerateChallengeClient.this,
false /* success */);
mCallback.onClientFinished(this, false /* success */);
}
}
}

View File

@@ -48,6 +48,7 @@ class FingerprintRevokeChallengeClient extends RevokeChallengeClient<ISession> {
getFreshDaemon().revokeChallenge(mChallenge);
} catch (RemoteException e) {
Slog.e(TAG, "Unable to revokeChallenge", e);
mCallback.onClientFinished(this, false /* success */);
}
}

View File

@@ -55,6 +55,7 @@ public class FingerprintGenerateChallengeClient
}
} catch (RemoteException e) {
Slog.e(TAG, "preEnroll failed", e);
mCallback.onClientFinished(this, false /* success */);
}
}
}

View File

@@ -45,8 +45,10 @@ public class FingerprintRevokeChallengeClient
protected void startHalOperation() {
try {
getFreshDaemon().postEnroll();
mCallback.onClientFinished(this, true /* success */);
} catch (RemoteException e) {
Slog.e(TAG, "revokeChallenge/postEnroll failed", e);
mCallback.onClientFinished(this, false /* success */);
}
}
}