From f27874740f02d8acba79f2d2924a62e9162da02b Mon Sep 17 00:00:00 2001 From: Nick Kralevich Date: Tue, 22 Jan 2013 12:44:11 -0800 Subject: [PATCH] Revert "libc: make system properties more secure." This reverts commit f10c5a2215b3da2e226e8bd148c86e2c146d8e90. Bug: 8045561 --- libc/bionic/system_properties.c | 29 +++++++++++++-------------- libc/include/sys/_system_properties.h | 1 - 2 files changed, 14 insertions(+), 16 deletions(-) diff --git a/libc/bionic/system_properties.c b/libc/bionic/system_properties.c index 8e53accfd..caa5ca61b 100644 --- a/libc/bionic/system_properties.c +++ b/libc/bionic/system_properties.c @@ -31,7 +31,6 @@ #include #include #include -#include #include @@ -39,7 +38,6 @@ #include #include #include -#include #include #include @@ -57,34 +55,33 @@ prop_area *__system_property_area__ = (void*) &dummy_props; int __system_properties_init(void) { prop_area *pa; - int fd; - struct stat fd_stat; + int s, fd; + unsigned sz; + char *env; if(__system_property_area__ != ((void*) &dummy_props)) { return 0; } - fd = open(PROP_FILENAME, O_RDONLY | O_NOFOLLOW); - - if (fd < 0) { + env = getenv("ANDROID_PROPERTY_WORKSPACE"); + if (!env) { return -1; } - - if (fstat(fd, &fd_stat) < 0) { - close(fd); + fd = atoi(env); + env = strchr(env, ','); + if (!env) { return -1; } - - pa = mmap(0, fd_stat.st_size, PROT_READ, MAP_SHARED, fd, 0); - - close(fd); + sz = atoi(env + 1); + + pa = mmap(0, sz, PROT_READ, MAP_SHARED, fd, 0); if(pa == MAP_FAILED) { return -1; } if((pa->magic != PROP_AREA_MAGIC) || (pa->version != PROP_AREA_VERSION)) { - munmap(pa, fd_stat.st_size); + munmap(pa, sz); return -1; } @@ -221,6 +218,8 @@ static int send_prop_msg(prop_msg *msg) int __system_property_set(const char *key, const char *value) { int err; + int tries = 0; + int update_seen = 0; prop_msg msg; if(key == 0) return -1; diff --git a/libc/include/sys/_system_properties.h b/libc/include/sys/_system_properties.h index 10c0fae16..42a7f6c0f 100644 --- a/libc/include/sys/_system_properties.h +++ b/libc/include/sys/_system_properties.h @@ -41,7 +41,6 @@ typedef struct prop_msg prop_msg; #define PROP_AREA_VERSION 0x45434f76 #define PROP_SERVICE_NAME "property_service" -#define PROP_FILENAME "/dev/__properties__" /* #define PROP_MAX_ENTRIES 247 */ /* 247 -> 32620 bytes (<32768) */