Commit 9aed3ce1 authored Jul 10, 2025 by Hakjun Choi

Throw IllegalStateException instead of RuntimeException when remote exception...

Throw IllegalStateException instead of RuntimeException when remote exception wraps IllegalStateException

IllegalStateException is explicitly described that it'll be thrown when FWK has issue, but there is a case it is thrown wrapped by RuntimeException. in this case, caller can't manage IllegalStateException.

Bug: 430494109
Flag: EXEMPT bug fix
Test: manually run device test with hard coded firmware and verified caller received thrown IllegalStateException
Change-Id: I707548a7601a85f26b151d3b9e06cb45f7b049aa

parent 3ad56027

telephony/java/android/telephony/satellite/SatelliteManager.java

+4 −0

Original line number	Diff line number	Diff line
		@@ -3123,8 +3123,12 @@ public final class SatelliteManager {
		} else {
		throw new IllegalStateException("Telephony service is null.");
		}
		} catch (IllegalStateException ex) {
		loge("registerForNtnSignalStrengthChanged() IllegalStateException: " + ex);
		throw ex;
		} catch (RemoteException ex) {
		loge("registerForNtnSignalStrengthChanged() RemoteException: " + ex);
		// Treat other remote exceptions as fatal system errors.
		ex.rethrowAsRuntimeException();
		}
		}