Revert "bionic: prevent root processes from calling __system_property_add"

This reverts commit fb9b7b436f3ef94385f1b0c55ab81f246f0d96b8.

diff --git a/libc/bionic/system_properties.c b/libc/bionic/system_properties.c
index f9671c6..d4054d2 100644 (file)
--- a/libc/bionic/system_properties.c
+++ b/libc/bionic/system_properties.c
@@ -128,13 +128,11 @@ static int map_prop_area_rw()
 {
     prop_area *pa;
     int fd;
-    int ret;
 
     /* dev is a tmpfs that we can use to carve a shared workspace
      * out of, so let's do that...
      */
-    fd = open(property_filename, O_RDWR | O_CREAT | O_NOFOLLOW | O_CLOEXEC |
-            O_EXCL, 0444);
+    fd = open(property_filename, O_RDWR | O_CREAT | O_NOFOLLOW, 0644);
     if (fd < 0) {
         if (errno == EACCES) {
             /* for consistency with the case where the process has already
@@ -145,10 +143,6 @@ static int map_prop_area_rw()
         return -1;
     }
 
-    ret = fcntl(fd, F_SETFD, FD_CLOEXEC);
-    if (ret < 0)
-        goto out;
-
     if (ftruncate(fd, PA_SIZE) < 0)
         goto out;
 
@@ -192,16 +186,8 @@ static int map_prop_area()
 {
     bool fromFile = true;
     int result = -1;
-    int fd;
-    int ret;
-
-    fd = open(property_filename, O_RDONLY | O_NOFOLLOW | O_CLOEXEC);
-    if (fd >= 0) {
-        /* For old kernels that don't support O_CLOEXEC */
-        ret = fcntl(fd, F_SETFD, FD_CLOEXEC);
-        if (ret < 0)
-            goto cleanup;
-    }
+
+    int fd = open(property_filename, O_RDONLY | O_NOFOLLOW);
 
     if ((fd < 0) && (errno == ENOENT)) {
         /*