commit 57d977500c1612adab3e461d0d56da803525f2ec
author Andreas Huber <andih@google.com> Fri Aug 18 14:43:52 2017 -0700
committer Andreas Huber <andih@google.com> Mon Aug 21 09:05:13 2017 -0700
tree f6881272819aa00b1bf9c0ef95dd0a9fced17fcb
parent 37612b1cdd0e65fae8c2f70f6be8f6847a64ee5c

Fix use-after-free of stack-allocated temporary string.

Bug: 64848081
Test: built and successfully booted again
Merged-In: I93c899249bf2cc5ab8d880c0eaff471518e73121

Change-Id: I93c899249bf2cc5ab8d880c0eaff471518e73121

init/selinux.cpp

diff --git a/init/selinux.cpp b/init/selinux.cpp
index 1104fb3..201c070 100644
--- a/init/selinux.cpp
+++ b/init/selinux.cpp
@@ -277,13 +277,15 @@
         return false;
     }
 
+    const std::string version_as_string = std::to_string(max_policy_version);
+
     // clang-format off
     const char* compile_args[] = {
         "/system/bin/secilc",
         plat_policy_cil_file,
         "-M", "true",
         // Target the highest policy language version supported by the kernel
-        "-c", std::to_string(max_policy_version).c_str(),
+        "-c", version_as_string.c_str(),
         "/system/etc/selinux/mapping_sepolicy.cil",
         "/vendor/etc/selinux/nonplat_sepolicy.cil",
         "-o", compiled_sepolicy,