diff options
Diffstat (limited to 'src/java/com')
-rw-r--r-- | src/java/com/android/ike/ikev2/ChildSessionCallback.java (renamed from src/java/com/android/ike/ikev2/IChildSessionCallback.java) | 6 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/ChildSessionStateMachine.java | 10 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/ChildSessionStateMachineFactory.java | 6 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/IkeLocalRequestScheduler.java | 4 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/IkeManager.java | 8 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/IkeSession.java | 18 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/IkeSessionCallback.java (renamed from src/java/com/android/ike/ikev2/IIkeSessionCallback.java) | 8 | ||||
-rw-r--r-- | src/java/com/android/ike/ikev2/IkeSessionStateMachine.java | 37 |
8 files changed, 49 insertions, 48 deletions
diff --git a/src/java/com/android/ike/ikev2/IChildSessionCallback.java b/src/java/com/android/ike/ikev2/ChildSessionCallback.java index 81f751dc..071ad432 100644 --- a/src/java/com/android/ike/ikev2/IChildSessionCallback.java +++ b/src/java/com/android/ike/ikev2/ChildSessionCallback.java @@ -22,7 +22,7 @@ import android.net.IpSecTransform; import com.android.ike.ikev2.exceptions.IkeException; /** Callback interface for receiving state changes of a Child Session. */ -public interface IChildSessionCallback { +public interface ChildSessionCallback { /** * Called when Child Session setup succeeds. * @@ -40,11 +40,11 @@ public interface IChildSessionCallback { void onClosed(); /** - * Called if Child Session setup fails or Child Session is deleted because of a fatal error. + * Called if Child Session setup fails or Child Session is closed because of a fatal error. * * @param exception the detailed error. */ - void onError(IkeException exception); + void onClosedExceptionally(IkeException exception); /** * Called when a new {@link IpSecTransform} is created for this Child Session. diff --git a/src/java/com/android/ike/ikev2/ChildSessionStateMachine.java b/src/java/com/android/ike/ikev2/ChildSessionStateMachine.java index 7e674ef2..0d591fee 100644 --- a/src/java/com/android/ike/ikev2/ChildSessionStateMachine.java +++ b/src/java/com/android/ike/ikev2/ChildSessionStateMachine.java @@ -145,7 +145,7 @@ public class ChildSessionStateMachine extends AbstractSessionStateMachine { private final ChildSessionOptions mChildSessionOptions; private final Executor mUserCbExecutor; - private final IChildSessionCallback mUserCallback; + private final ChildSessionCallback mUserCallback; /** Callback to notify IKE Session the state changes. */ private final IChildSessionSmCallback mChildSmCallback; @@ -219,7 +219,7 @@ public class ChildSessionStateMachine extends AbstractSessionStateMachine { IpSecManager ipSecManager, ChildSessionOptions sessionOptions, Executor userCbExecutor, - IChildSessionCallback userCallback, + ChildSessionCallback userCallback, IChildSessionSmCallback childSmCallback) { super(TAG, looper); @@ -293,7 +293,7 @@ public class ChildSessionStateMachine extends AbstractSessionStateMachine { * <p>This method MUST be called after the user callbacks have been fired, and MUST always * be called before the state machine can shut down. */ - void onChildSessionClosed(IChildSessionCallback userCallbacks); + void onChildSessionClosed(ChildSessionCallback userCallbacks); /** * Notify that a Child procedure has been finished and the IKE Session should close itself @@ -577,7 +577,7 @@ public class ChildSessionStateMachine extends AbstractSessionStateMachine { mUserCbExecutor.execute( () -> { - mUserCallback.onError(new IkeInternalException(e)); + mUserCallback.onClosedExceptionally(new IkeInternalException(e)); }); logWtf("Unexpected exception in " + getCurrentState().getName(), e); quitNow(); @@ -636,7 +636,7 @@ public class ChildSessionStateMachine extends AbstractSessionStateMachine { mUserCbExecutor.execute( () -> { - mUserCallback.onError(ikeException); + mUserCallback.onClosedExceptionally(ikeException); }); loge("Child Session fatal error", ikeException); diff --git a/src/java/com/android/ike/ikev2/ChildSessionStateMachineFactory.java b/src/java/com/android/ike/ikev2/ChildSessionStateMachineFactory.java index 53c7d81f..dc68e9f1 100644 --- a/src/java/com/android/ike/ikev2/ChildSessionStateMachineFactory.java +++ b/src/java/com/android/ike/ikev2/ChildSessionStateMachineFactory.java @@ -37,7 +37,7 @@ final class ChildSessionStateMachineFactory { Context context, ChildSessionOptions sessionOptions, Executor userCbExecutor, - IChildSessionCallback userCallbacks, + ChildSessionCallback userCallbacks, IChildSessionSmCallback childSmCallback) { return sChildSessionHelper.makeChildSessionStateMachine( looper, context, sessionOptions, userCbExecutor, userCallbacks, childSmCallback); @@ -60,7 +60,7 @@ final class ChildSessionStateMachineFactory { Context context, ChildSessionOptions sessionOptions, Executor userCbExecutor, - IChildSessionCallback userCallbacks, + ChildSessionCallback userCallbacks, IChildSessionSmCallback childSmCallback); } @@ -75,7 +75,7 @@ final class ChildSessionStateMachineFactory { Context context, ChildSessionOptions sessionOptions, Executor userCbExecutor, - IChildSessionCallback userCallbacks, + ChildSessionCallback userCallbacks, IChildSessionSmCallback childSmCallback) { ChildSessionStateMachine childSession = new ChildSessionStateMachine( diff --git a/src/java/com/android/ike/ikev2/IkeLocalRequestScheduler.java b/src/java/com/android/ike/ikev2/IkeLocalRequestScheduler.java index a1447313..5039f2a4 100644 --- a/src/java/com/android/ike/ikev2/IkeLocalRequestScheduler.java +++ b/src/java/com/android/ike/ikev2/IkeLocalRequestScheduler.java @@ -93,11 +93,11 @@ public final class IkeLocalRequestScheduler { * initiated locally. */ public static class ChildLocalRequest extends LocalRequest { - public final IChildSessionCallback childSessionCallback; + public final ChildSessionCallback childSessionCallback; public final ChildSessionOptions childSessionOptions; ChildLocalRequest( - int type, IChildSessionCallback childCallback, ChildSessionOptions childOptions) { + int type, ChildSessionCallback childCallback, ChildSessionOptions childOptions) { super(type); childSessionOptions = childOptions; childSessionCallback = childCallback; diff --git a/src/java/com/android/ike/ikev2/IkeManager.java b/src/java/com/android/ike/ikev2/IkeManager.java index 446ce281..38ab15ba 100644 --- a/src/java/com/android/ike/ikev2/IkeManager.java +++ b/src/java/com/android/ike/ikev2/IkeManager.java @@ -54,9 +54,9 @@ public final class IkeManager { * @param userCbExecutor the {@link Executor} upon which all callbacks will be posted. For * security and consistency, the callbacks posted to this executor MUST be executed * serially, in the order they were posted. - * @param ikeSessionCallback the {@link IIkeSessionCallback} interface to notify users the state + * @param ikeSessionCallback the {@link IkeSessionCallback} interface to notify users the state * changes of the IKE Session. - * @param firstChildSessionCallback the {@link IChildSessionCallback} interface to notify users + * @param firstChildSessionCallback the {@link ChildSessionCallback} interface to notify users * the state changes of the Child Session. * @return an instance of {@link IkeSession} */ @@ -64,8 +64,8 @@ public final class IkeManager { IkeSessionOptions ikeSessionOptions, ChildSessionOptions firstChildSessionOptions, Executor userCbExecutor, - IIkeSessionCallback ikeSessionCallback, - IChildSessionCallback firstChildSessionCallback) { + IkeSessionCallback ikeSessionCallback, + ChildSessionCallback firstChildSessionCallback) { return new IkeSession( mContext, ikeSessionOptions, diff --git a/src/java/com/android/ike/ikev2/IkeSession.java b/src/java/com/android/ike/ikev2/IkeSession.java index 2147471a..19d9cb96 100644 --- a/src/java/com/android/ike/ikev2/IkeSession.java +++ b/src/java/com/android/ike/ikev2/IkeSession.java @@ -38,8 +38,8 @@ public final class IkeSession implements AutoCloseable { IkeSessionOptions ikeSessionOptions, ChildSessionOptions firstChildSessionOptions, Executor userCbExecutor, - IIkeSessionCallback ikeSessionCallback, - IChildSessionCallback firstChildSessionCallback) { + IkeSessionCallback ikeSessionCallback, + ChildSessionCallback firstChildSessionCallback) { this( IkeThreadHolder.IKE_WORKER_THREAD.getLooper(), context, @@ -60,8 +60,8 @@ public final class IkeSession implements AutoCloseable { IkeSessionOptions ikeSessionOptions, ChildSessionOptions firstChildSessionOptions, Executor userCbExecutor, - IIkeSessionCallback ikeSessionCallback, - IChildSessionCallback firstChildSessionCallback) { + IkeSessionCallback ikeSessionCallback, + ChildSessionCallback firstChildSessionCallback) { mIkeSessionStateMachine = new IkeSessionStateMachine( looper, @@ -99,17 +99,17 @@ public final class IkeSession implements AutoCloseable { /** * Initiate Create Child exchange on the IKE worker thread. * - * <p>Users MUST provide a unique {@link IChildSessionCallback} instance for each new Child + * <p>Users MUST provide a unique {@link ChildSessionCallback} instance for each new Child * Session. * * @param childSessionOptions the {@link ChildSessionOptions} that contains the Child Session * configurations to negotiate. - * @param childSessionCallback the {@link IChildSessionCallback} interface to notify users the + * @param childSessionCallback the {@link ChildSessionCallback} interface to notify users the * state changes of the Child Session. - * @throws IllegalArgumentException if the IChildSessionCallback is already in use. + * @throws IllegalArgumentException if the ChildSessionCallback is already in use. */ public void openChildSession( - ChildSessionOptions childSessionOptions, IChildSessionCallback childSessionCallback) { + ChildSessionOptions childSessionOptions, ChildSessionCallback childSessionCallback) { mIkeSessionStateMachine.openChildSession(childSessionOptions, childSessionCallback); } @@ -120,7 +120,7 @@ public final class IkeSession implements AutoCloseable { * interface to notify users the deletion result. * @throws IllegalArgumentException if no Child Session found bound with this callback. */ - public void closeChildSession(IChildSessionCallback childSessionCallback) { + public void closeChildSession(ChildSessionCallback childSessionCallback) { mIkeSessionStateMachine.closeChildSession(childSessionCallback); } diff --git a/src/java/com/android/ike/ikev2/IIkeSessionCallback.java b/src/java/com/android/ike/ikev2/IkeSessionCallback.java index ee196797..66081206 100644 --- a/src/java/com/android/ike/ikev2/IIkeSessionCallback.java +++ b/src/java/com/android/ike/ikev2/IkeSessionCallback.java @@ -21,7 +21,7 @@ import android.annotation.NonNull; import com.android.ike.ikev2.exceptions.IkeException; /** Callback interface for receiving state changes of an IKE Session. */ -public interface IIkeSessionCallback { +public interface IkeSessionCallback { /** * Called when negotiation and authentication for this new IKE Session succeeds. * @@ -39,11 +39,11 @@ public interface IIkeSessionCallback { void onClosed(); /** - * Called if IKE Session negotiation fails or IKE Session is deleted because of a fatal error. + * Called if IKE Session negotiation fails or IKE Session is closed because of a fatal error. * * @param exception the detailed error. */ - void onError(IkeException exception); + void onClosedExceptionally(IkeException exception); /** * Called if a recoverable error is encountered in an established IKE Session. @@ -54,5 +54,5 @@ public interface IIkeSessionCallback { * * @param exception the detailed error. */ - void onInfo(IkeException exception); + void onError(IkeException exception); } diff --git a/src/java/com/android/ike/ikev2/IkeSessionStateMachine.java b/src/java/com/android/ike/ikev2/IkeSessionStateMachine.java index c0893254..7172dace 100644 --- a/src/java/com/android/ike/ikev2/IkeSessionStateMachine.java +++ b/src/java/com/android/ike/ikev2/IkeSessionStateMachine.java @@ -281,13 +281,13 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { private final IpSecManager mIpSecManager; private final IkeLocalRequestScheduler mScheduler; private final Executor mUserCbExecutor; - private final IIkeSessionCallback mIkeSessionCallback; + private final IkeSessionCallback mIkeSessionCallback; private final IkeEapAuthenticatorFactory mEapAuthenticatorFactory; private final TempFailureHandler mTempFailHandler; @VisibleForTesting @GuardedBy("mChildCbToSessions") - final HashMap<IChildSessionCallback, ChildSessionStateMachine> mChildCbToSessions = + final HashMap<ChildSessionCallback, ChildSessionStateMachine> mChildCbToSessions = new HashMap<>(); /** @@ -334,7 +334,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { // FIXME: b/131265898 Move into CreateIkeLocalIkeAuth, and pass through to // CreateIkeLocalIkeAuthPostEap once passing entry data is supported private ChildSessionOptions mFirstChildSessionOptions; - private IChildSessionCallback mFirstChildCallbacks; + private ChildSessionCallback mFirstChildCallbacks; /** Package */ @VisibleForTesting IkeSaRecord mCurrentIkeSaRecord; @@ -384,8 +384,8 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { IkeSessionOptions ikeOptions, ChildSessionOptions firstChildOptions, Executor userCbExecutor, - IIkeSessionCallback ikeSessionCallback, - IChildSessionCallback firstChildSessionCallback, + IkeSessionCallback ikeSessionCallback, + ChildSessionCallback firstChildSessionCallback, IkeEapAuthenticatorFactory eapAuthenticatorFactory) { super(TAG, looper); @@ -443,8 +443,8 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { IkeSessionOptions ikeOptions, ChildSessionOptions firstChildOptions, Executor userCbExecutor, - IIkeSessionCallback ikeSessionCallback, - IChildSessionCallback firstChildSessionCallback) { + IkeSessionCallback ikeSessionCallback, + ChildSessionCallback firstChildSessionCallback) { this( looper, context, @@ -457,7 +457,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { new IkeEapAuthenticatorFactory()); } - private boolean hasChildSessionCallback(IChildSessionCallback callback) { + private boolean hasChildSessionCallback(ChildSessionCallback callback) { synchronized (mChildCbToSessions) { return mChildCbToSessions.containsKey(callback); } @@ -478,7 +478,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { @VisibleForTesting void registerChildSessionCallback( ChildSessionOptions childOptions, - IChildSessionCallback callbacks, + ChildSessionCallback callbacks, boolean isFirstChild) { synchronized (mChildCbToSessions) { if (!isFirstChild && getCurrentState() == null) { @@ -503,7 +503,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { } void openChildSession( - ChildSessionOptions childSessionOptions, IChildSessionCallback childSessionCallback) { + ChildSessionOptions childSessionOptions, ChildSessionCallback childSessionCallback) { if (childSessionCallback == null) { throw new IllegalArgumentException("Child Session Callback must be provided"); } @@ -520,7 +520,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { CMD_LOCAL_REQUEST_CREATE_CHILD, childSessionCallback, childSessionOptions)); } - void closeChildSession(IChildSessionCallback childSessionCallback) { + void closeChildSession(ChildSessionCallback childSessionCallback) { if (childSessionCallback == null) { throw new IllegalArgumentException("Child Session Callback must be provided"); } @@ -576,7 +576,8 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { + " of sync."); mUserCbExecutor.execute( () -> { - mIkeSessionCallback.onError(new IkeInternalException(error)); + mIkeSessionCallback.onClosedExceptionally( + new IkeInternalException(error)); }); loge("Fatal error", error); @@ -757,13 +758,13 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { /** Class to group parameters for negotiating the first Child SA. */ private static class FirstChildNegotiationData { public final ChildSessionOptions childSessionOptions; - public final IChildSessionCallback childSessionCallback; + public final ChildSessionCallback childSessionCallback; public final List<IkePayload> reqPayloads; public final List<IkePayload> respPayloads; FirstChildNegotiationData( ChildSessionOptions childSessionOptions, - IChildSessionCallback childSessionCallback, + ChildSessionCallback childSessionCallback, List<IkePayload> reqPayloads, List<IkePayload> respPayloads) { this.childSessionOptions = childSessionOptions; @@ -838,7 +839,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { } @Override - public void onChildSessionClosed(IChildSessionCallback userCallbacks) { + public void onChildSessionClosed(ChildSessionCallback userCallbacks) { synchronized (mChildCbToSessions) { mChildCbToSessions.remove(userCallbacks); } @@ -860,7 +861,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { mUserCbExecutor.execute( () -> { - mIkeSessionCallback.onError(new IkeInternalException(e)); + mIkeSessionCallback.onClosedExceptionally(new IkeInternalException(e)); }); logWtf("Unexpected exception in " + getCurrentState().getName(), e); quitNow(); @@ -924,7 +925,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { closeAllSaRecords(false /*expectSaClosed*/); mUserCbExecutor.execute( () -> { - mIkeSessionCallback.onError(ikeException); + mIkeSessionCallback.onClosedExceptionally(ikeException); }); loge("IKE Session fatal error in " + getCurrentState().getName(), ikeException); @@ -1970,7 +1971,7 @@ public class IkeSessionStateMachine extends AbstractSessionStateMachine { } } - private ChildSessionStateMachine getChildSession(IChildSessionCallback callbacks) { + private ChildSessionStateMachine getChildSession(ChildSessionCallback callbacks) { synchronized (mChildCbToSessions) { return mChildCbToSessions.get(callbacks); } |