From 0d375e69e15fd252629360712d9dfd80cfb3ac93 Mon Sep 17 00:00:00 2001
From: Wiktor Garbacz <wiktorg@google.com>
Date: Wed, 1 Jul 2020 04:53:52 -0700
Subject: [PATCH] Remove abort from ExecuteProcess

Otherwise ExecuteProcess is implicitly `[[noreturn]]` and this
might cause policy violations in `__asan_handle_no_return`
for ASAN builds.

PiperOrigin-RevId: 319203128
Change-Id: I5c8ba71ce88261f803aa3f16730eccea0d803dd1
---
 sandboxed_api/sandbox2/forkserver.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/sandboxed_api/sandbox2/forkserver.cc b/sandboxed_api/sandbox2/forkserver.cc
index 7b09a1a..0998714 100644
--- a/sandboxed_api/sandbox2/forkserver.cc
+++ b/sandboxed_api/sandbox2/forkserver.cc
@@ -624,7 +624,6 @@ void ForkServer::ExecuteProcess(int execve_fd, const char** argv,
   }
 
   util::Syscall(__NR_exit_group, EXIT_FAILURE);
-  abort();
 }
 
 void ForkServer::InitializeNamespaces(const ForkRequest& request, uid_t uid,