diff options
author | Kenny Root <kroot@google.com> | 2012-09-13 10:52:52 -0700 |
---|---|---|
committer | Kenny Root <kroot@google.com> | 2012-09-13 15:25:09 -0700 |
commit | 2a54e5ecd0a96398e8d7d9b1629ecf8fb1633a2b (patch) | |
tree | 0fa9a13009d0f9e215921fa3bc58b8b8b23d158c /libc/bionic/stubs.cpp | |
parent | fa36875df423af968d053c584c0de18a2448516b (diff) | |
download | bionic-2a54e5ecd0a96398e8d7d9b1629ecf8fb1633a2b.zip bionic-2a54e5ecd0a96398e8d7d9b1629ecf8fb1633a2b.tar.gz bionic-2a54e5ecd0a96398e8d7d9b1629ecf8fb1633a2b.tar.bz2 |
Add tests for Android-specific stubs
Also fix problem with multi-user IDs that the home directory was
returned as "/data" instead of "/" unlike all the other uids.
Change-Id: I914d22052e5a86552989f8969b85aadbc748c65d
Diffstat (limited to 'libc/bionic/stubs.cpp')
-rw-r--r-- | libc/bionic/stubs.cpp | 30 |
1 files changed, 21 insertions, 9 deletions
diff --git a/libc/bionic/stubs.cpp b/libc/bionic/stubs.cpp index a539b34..fb1a8e4 100644 --- a/libc/bionic/stubs.cpp +++ b/libc/bionic/stubs.cpp @@ -264,10 +264,8 @@ static unsigned app_id_from_name(const char* name) { return (unsigned)(appid + userid*AID_USER); } -static void print_app_uid_name(uid_t uid, char* buffer, int bufferlen) { - uid_t appid = uid % AID_USER; - uid_t userid = uid / AID_USER; - +static void print_app_name_from_appid_userid(const uid_t appid, + const uid_t userid, char* buffer, const int bufferlen) { if (appid < AID_ISOLATED_START) { if (appid < AID_APP) { for (size_t n = 0; n < android_id_count; n++) { @@ -283,6 +281,12 @@ static void print_app_uid_name(uid_t uid, char* buffer, int bufferlen) { } } +static void print_app_name_from_uid(const uid_t uid, char* buffer, const int bufferlen) { + const uid_t appid = uid % AID_USER; + const uid_t userid = uid / AID_USER; + return print_app_name_from_appid_userid(appid, userid, buffer, bufferlen); +} + // Translate a uid into the corresponding name. // 0 to AID_APP-1 -> "system", "radio", etc. // AID_APP to AID_ISOLATED_START-1 -> u0_a1234 @@ -297,10 +301,18 @@ static passwd* app_id_to_passwd(uid_t uid, stubs_state_t* state) { return NULL; } - print_app_uid_name(uid, state->app_name_buffer_, - sizeof(state->app_name_buffer_)); + const uid_t appid = uid % AID_USER; + const uid_t userid = uid / AID_USER; + + print_app_name_from_appid_userid(appid, userid, state->app_name_buffer_, + sizeof(state->app_name_buffer_)); + + if (appid < AID_APP) { + snprintf(state->dir_buffer_, sizeof(state->dir_buffer_), "/"); + } else { + snprintf(state->dir_buffer_, sizeof(state->dir_buffer_), "/data"); + } - snprintf(state->dir_buffer_, sizeof(state->dir_buffer_), "/data"); snprintf(state->sh_buffer_, sizeof(state->sh_buffer_), "/system/bin/sh"); pw->pw_name = state->app_name_buffer_; @@ -320,8 +332,8 @@ static group* app_id_to_group(gid_t gid, stubs_state_t* state) { return NULL; } - print_app_uid_name(gid, state->group_name_buffer_, - sizeof(state->group_name_buffer_)); + print_app_name_from_uid(gid, state->group_name_buffer_, + sizeof(state->group_name_buffer_)); group* gr = &state->group_; gr->gr_name = state->group_name_buffer_; |