diff options
author | Mathias Agopian <mathias@google.com> | 2009-06-15 18:24:59 -0700 |
---|---|---|
committer | Mathias Agopian <mathias@google.com> | 2009-06-15 21:56:51 -0700 |
commit | 375f56363a8737119ce2222dcfaacbe1cf733fc0 (patch) | |
tree | 5cdf2be415ef2668b6d4179b00b61e5a57f5c077 /include | |
parent | 5e78e0965169790111f01354e78b0f8d34c94840 (diff) | |
download | frameworks_native-375f56363a8737119ce2222dcfaacbe1cf733fc0.zip frameworks_native-375f56363a8737119ce2222dcfaacbe1cf733fc0.tar.gz frameworks_native-375f56363a8737119ce2222dcfaacbe1cf733fc0.tar.bz2 |
new Permission class used to improve permission checks speed (by caching results)
Diffstat (limited to 'include')
-rw-r--r-- | include/binder/IServiceManager.h | 2 | ||||
-rw-r--r-- | include/binder/Permission.h | 68 |
2 files changed, 70 insertions, 0 deletions
diff --git a/include/binder/IServiceManager.h b/include/binder/IServiceManager.h index ea149dd..24e9e99 100644 --- a/include/binder/IServiceManager.h +++ b/include/binder/IServiceManager.h @@ -78,6 +78,8 @@ status_t getService(const String16& name, sp<INTERFACE>* outService) bool checkCallingPermission(const String16& permission); bool checkCallingPermission(const String16& permission, int32_t* outPid, int32_t* outUid); +bool checkPermission(const String16& permission, pid_t pid, uid_t uid); + // ---------------------------------------------------------------------- diff --git a/include/binder/Permission.h b/include/binder/Permission.h new file mode 100644 index 0000000..9542d50 --- /dev/null +++ b/include/binder/Permission.h @@ -0,0 +1,68 @@ +/* + * Copyright (C) 2009 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#ifndef BINDER_PERMISSION_H +#define BINDER_PERMISSION_H + +#include <stdint.h> +#include <unistd.h> + +#include <utils/SortedVector.h> +#include <utils/String16.h> +#include <utils/threads.h> + +namespace android { +// --------------------------------------------------------------------------- + +/* + * Permission caches the result of the permission check for the given + * permission name and the provided uid/pid. It also handles a few + * known cases efficiently (caller is in the same process or is root). + * The package manager does something similar but lives in dalvik world + * and is therefore extremely slow to access. + */ + +class Permission +{ +public: + Permission(char const* name); + Permission(const String16& name); + Permission(const Permission& rhs); + virtual ~Permission(); + + bool operator < (const Permission& rhs) const; + + // checks the current binder call's caller has access to this permission + bool checkCalling() const; + + // checks the specified pid/uid has access to this permission + bool check(pid_t pid, uid_t uid) const; + +protected: + virtual bool doCheckPermission(pid_t pid, uid_t uid) const; + +private: + Permission& operator = (const Permission& rhs) const; + const String16 mPermissionName; + mutable SortedVector<uid_t> mGranted; + const pid_t mPid; + mutable Mutex mLock; +}; + +// --------------------------------------------------------------------------- +}; // namespace android + +#endif /* BINDER_PERMISSION_H */ |