diff options
author | Nick Chalko <nchalko@google.com> | 2021-04-28 16:23:55 -0700 |
---|---|---|
committer | Nick Chalko <nchalko@google.com> | 2021-04-28 16:53:05 -0700 |
commit | 928bbba890784eb97a4fca43c89245724f4773be (patch) | |
tree | 6cf63bde64b030cd56ed04a4867564f49d15d412 /src/com | |
parent | 80fac3bfc7f6fc224ef63fdb73e00b4233db3458 (diff) | |
download | TV-928bbba890784eb97a4fca43c89245724f4773be.tar.gz |
Check for null calling Activity in SetupPassthroughActivity
This prevents by passing the calling Activity name check.
Bug: 174046975
Test: atest SetupPassthroughActivityTest
Change-Id: I9438d687f1c60a12d8839b619e9b47ac928315fe
Diffstat (limited to 'src/com')
-rw-r--r-- | src/com/android/tv/SetupPassthroughActivity.java | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/src/com/android/tv/SetupPassthroughActivity.java b/src/com/android/tv/SetupPassthroughActivity.java index 25049f1d..e7f89108 100644 --- a/src/com/android/tv/SetupPassthroughActivity.java +++ b/src/com/android/tv/SetupPassthroughActivity.java @@ -118,13 +118,12 @@ public class SetupPassthroughActivity extends Activity { setupIntent.putExtras(extras); try { ComponentName callingActivity = getCallingActivity(); - if (callingActivity != null - && !callingActivity.getPackageName().equals(CommonConstants.BASE_PACKAGE)) { - Log.w( - TAG, - "Calling activity " - + callingActivity.getPackageName() - + " is not trusted. Not forwarding intent."); + if (callingActivity == null + || !callingActivity.getPackageName().equals(CommonConstants.BASE_PACKAGE)) { + String name = + callingActivity == null ? "null" : callingActivity.getPackageName(); + Log.w(TAG, + "Calling activity " + name + " is not trusted. Not forwarding intent."); finish(); return; } |