commit 2a3fa67c8294c6108cf368be65de2658642ac349
author Calin Juravle <calin@google.com> Thu May 23 13:14:35 2019 -0700
committer Nikoli Cartagena <dargeren@google.com> Mon Jun 10 15:02:13 2019 -0700
tree cd88d6d219d8b1570e89614e3b1062d852a6935d
parent b151df35115c546fd72107de43dd5e3a4a7f6bd4

Use conservative permissions when creating files in ART

Use 644 permissions by default so that only the owner has write-rights.

Bug: 130821293
Test: host run-test & gtest
Merged-In: I09eab9749d1ce5872ecaf7d5a80fc45c9c708853
Change-Id: Ic4dae399a5bfe862aff3d8614c45b38044d805db
(cherry picked from commit d224e964bd0de231c72008547b0246a2f70048ee)

libartbase/base/os_linux.cc

diff --git a/libartbase/base/os_linux.cc b/libartbase/base/os_linux.cc
index cb228bd..c972f8f 100644
--- a/libartbase/base/os_linux.cc
+++ b/libartbase/base/os_linux.cc
@@ -57,7 +57,8 @@
   CHECK(name != nullptr);
   bool read_only = ((flags & O_ACCMODE) == O_RDONLY);
   bool check_usage = !read_only && auto_flush;
-  std::unique_ptr<File> file(new File(name, flags, 0666, check_usage));
+  std::unique_ptr<File> file(
+      new File(name, flags, S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH, check_usage));
   if (!file->IsOpened()) {
     return nullptr;
   }