joy/android_packages_modules_Connectivity
Watch
1
Fork
You've already forked android_packages_modules_Connectivity
 0
Code Issues Pull requests Projects Releases Wiki Activity

introduce mapRetrieveLocklessRW() am: 0fff839ae6

Browse source 
Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/3134554

Change-Id: I865908cecfaa37dca1959e9174065fcda756b964
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Maciej Żenczykowski 2024-06-15 12:45:19 +00:00 • committed by Automerger Merge Worker
commit e64933fecc
2 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

3
service/jni/com_android_server_connectivity_ClatCoordinator.cpp
View file

@ -80,7 +80,8 @@ static void verifyPerms(const char * const path,
case VERIFY_BIN: return;
case VERIFY_PROG: fd = bpf::retrieveProgram(path); break;
case VERIFY_MAP_RO: fd = bpf::mapRetrieveRO(path); break;
case VERIFY_MAP_RW: fd = bpf::mapRetrieveRW(path); break;
// lockless: we're just checking access rights and will immediately close the fd
case VERIFY_MAP_RW: fd = bpf::mapRetrieveLocklessRW(path); break;
}
if (fd < 0) ALOGF("bpf_obj_get '%s' failed, errno=%d", path, errno);

11
staticlibs/native/bpf_syscall_wrappers/include/BpfSyscallWrappers.h
View file

@ -166,11 +166,16 @@ inline int bpfLock(int fd, short type) {
return ret; // most likely -1 with errno == EAGAIN, due to already held lock
}
inline int mapRetrieveExclusiveRW(const char* pathname) {
return bpfLock(bpfFdGet(pathname, 0), F_WRLCK);
inline int mapRetrieveLocklessRW(const char* pathname) {
return bpfFdGet(pathname, 0);
}
inline int mapRetrieveExclusiveRW(const char* pathname) {
return bpfLock(mapRetrieveLocklessRW(pathname), F_WRLCK);
}
inline int mapRetrieveRW(const char* pathname) {
return bpfLock(bpfFdGet(pathname, 0), F_RDLCK);
return bpfLock(mapRetrieveLocklessRW(pathname), F_RDLCK);
}
inline int mapRetrieveRO(const char* pathname) {