Avoid pointless exception wrapping.

SpawnRunner#exec is already allowed to return a ForbiddenActionInputException, which is converted into a UserExecException in AbstractSpawnStrategy#exec. The wrapping provides no useful context, as this error occurs during input prefetching, which is independent from sandboxing.

PiperOrigin-RevId: 662558149
Change-Id: Ifd6bf078f73f5089661a0b570c40a20e8e42b41d

src/main/java/com/google/devtools/build/lib/sandbox/AbstractSandboxSpawnRunner.java

@@ -91,7 +91,7 @@ public AbstractSandboxSpawnRunner(CommandEnvironment cmdEnv) {
 
   @Override
   public final SpawnResult exec(Spawn spawn, SpawnExecutionContext context)
-      throws ExecException, InterruptedException {
+      throws ExecException, InterruptedException, ForbiddenActionInputException {
     ActionExecutionMetadata owner = spawn.getResourceOwner();
     context.report(SpawnSchedulingEvent.create(getName()));
 
@@ -116,11 +116,6 @@ public final SpawnResult exec(Spawn spawn, SpawnExecutionContext context)
           SandboxHelpers.createFailureDetail(
               "I/O exception during sandboxed execution", Code.EXECUTION_IO_EXCEPTION);
       throw new UserExecException(e, failureDetail);
-    } catch (ForbiddenActionInputException e) {
-      FailureDetail failureDetail =
-          SandboxHelpers.createFailureDetail(
-              "Forbidden input found during sandboxed execution", Code.FORBIDDEN_INPUT);
-      throw new UserExecException(e, failureDetail);
     }
   }