gh-152936: Make privileged functions available on Android#152977
gh-152936: Make privileged functions available on Android#152977sabamdarif wants to merge 1 commit into
Conversation
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
it's my first pr here so i don't know much about the NEWS entry, if it's not a problem can anyone tell me if i do need to add any NEWS entry ? |
Documentation build overview
|
thanks for the PR, as per my thought yes, this will need NEWS entry but you can wait for the review and then you can add NEWS entry ( to get 100 % sure 😀) |
|
i have another question for the Android test failure should i update the test_posix.py ? to do something like |
|
why did the Tests / Sanitizers / TSan (free-threading) (pull_request) failed i don't get it ? |
os.chroot,initgroups,setegid,seteuid,setgid.... are currently blocked on Android.
Because the underlying C functions crash the process when called by a non-root user on android, rather than returning an error.
So i added getuid() != 0 check (under #ifdef ANDROID) before each call, so it will return PermissionError for non-root just like normal Unix