Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Crash on pixels with Android 8.1 and any devices with Android P in sys_startgui #79

Closed
evabishchevich opened this issue May 21, 2018 · 3 comments

Comments

@evabishchevich
Copy link

evabishchevich commented May 21, 2018

There is a problem with Pixel XL / Pixel 2 XL / Pixel / Pixel 2 after last Android security update.

During initialization process in https://github.com/libpd/libpd project there are function calls libpd_init -> sys_startgui -> setuid. Calling setuid is prohibited in Android starting Oreo.
Looks like it was already fixed in libpd/libpd@b8b972a#diff-77187871d46f57651997feb0748feffdL90.
The same behaviour was detected on Android P Preview.

Stacktrace:

#00  pc 0000000000054bc8  /system/lib/libc.so (setuid+12)
  #01  pc 000000000004c343  /data/app/SOMEAPP/lib/arm/libpd.so (sys_startgui+1094)
  #02  pc 0000000000069b21  /data/app/SOMEAPP/lib/arm/libpd.so (libpd_init+152)
  #03  pc 000000000006a4ff  /data/app/SOMEAPP/lib/arm/libpd.so (libpd_queued_init+142)
  #04  pc 0000000000003b9d  /data/app/SOMEAPP/lib/arm/libpdnativeopensl.so (Java_org_puredata_core_PdBase_initialize+4)
@evabishchevich evabishchevich changed the title Crash on Pixels with Android 8.1 in sys_startgui Crash on pixels with Android 8.1 and any devices with Android P in sys_startgui Jun 25, 2018
@tkirshboim
Copy link
Member

tkirshboim commented Jun 29, 2018

@evabishchevich Could you test my pd_0.48 branch to see if it solves the issue?
#70

@tkirshboim
Copy link
Member

Should be now fixed through #70

@evabishchevich
Copy link
Author

@tkirshboim I've tested it with pd 0.48.1 and it was fixed. I guess 0.48 fixed the issue too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants