Merge "Revert "Relax neverallows for vendor to use /system/bin/sh"" into main am: bbff9f5ea1

Original change: https://android-review.googlesource.com/c/platform/system/sepolicy/+/2967124

Change-Id: I41a1e6a99a393eaada22f0df0ce0a86b90200d11
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

public/domain.te

@@ -924,9 +924,6 @@ full_treble_only(`
         -crash_dump_exec
         -netutils_wrapper_exec
         userdebug_or_eng(`-tcpdump_exec')
-        # Vendor components still can invoke shell commands via /system/bin/sh
-        -shell_exec
-        -toolbox_exec
     }:file { entrypoint execute execute_no_trans };
 ')
 
@@ -1007,9 +1004,6 @@ full_treble_only(`
     -task_profiles_api_file
     -task_profiles_file
     userdebug_or_eng(`-tcpdump_exec')
-    # Vendor components still can invoke shell commands via /system/bin/sh
-    -shell_exec
-    -toolbox_exec
   }:file *;
 ')
 

public/hal_neverallows.te

@@ -85,13 +85,7 @@ neverallow {
   halserverdomain
   -hal_dumpstate_server
   -hal_telephony_server
-} {
-  file_type
-  fs_type
-  # May invoke shell commands via /system/bin/sh
-  -shell_exec
-  -toolbox_exec
-}:file execute_no_trans;
+} { file_type fs_type }:file execute_no_trans;
 # Do not allow a process other than init to transition into a HAL domain.
 neverallow { domain -init } halserverdomain:process transition;
 # Only allow transitioning to a domain by running its executable. Do not