From 7b4d4f2aa0c08ea22917fe26da1cd81cd965cb69 Mon Sep 17 00:00:00 2001
From: Mayank Garg <gargmayank@google.com>
Date: Wed, 16 Nov 2022 12:34:49 -0800
Subject: Added permission requirement to OEM service implementation

- Also added a CTS test to check it

Bug: 245982224
Test: atest OemCarServiceImplTest
Change-Id: I4fd32d41780033a52167ee7d5368efb8ab6f58d7
Merged-In: I4fd32d41780033a52167ee7d5368efb8ab6f58d7
(cherry pick from commit 57b987d52a1bc64e35642626125a8256705510e7)
---
 car-lib/src/android/car/oem/OemCarService.java | 4 ++++
 tests/OemCarServiceTestApp/AndroidManifest.xml | 1 +
 2 files changed, 5 insertions(+)

diff --git a/car-lib/src/android/car/oem/OemCarService.java b/car-lib/src/android/car/oem/OemCarService.java
index 90c40575d6..de62ab6786 100644
--- a/car-lib/src/android/car/oem/OemCarService.java
+++ b/car-lib/src/android/car/oem/OemCarService.java
@@ -42,6 +42,10 @@ import java.util.Map;
  * This code will be running as part of the OEM Service. This provides basic implementation for OEM
  * Service. OEMs should extend this class and override relevant methods.
  *
+ * <p>
+ * OEM service implementation should have {@code android.car.permission.BIND_OEM_CAR_SERVICE} as
+ * required permission in manifest to connect to the OEM service.
+ *
  * @hide
  */
 @SystemApi
diff --git a/tests/OemCarServiceTestApp/AndroidManifest.xml b/tests/OemCarServiceTestApp/AndroidManifest.xml
index 8fdac76e39..98510155be 100644
--- a/tests/OemCarServiceTestApp/AndroidManifest.xml
+++ b/tests/OemCarServiceTestApp/AndroidManifest.xml
@@ -20,6 +20,7 @@
     <application>
         <service android:name="com.android.car.oemcarservice.testapp.OemCarServiceImpl"
                  android:directBootAware="true"
+                 android:permission="android.car.permission.BIND_OEM_CAR_SERVICE"
                  android:exported="true">
         </service>
     </application>
-- 
cgit v1.2.3