SIGSEGV in "am" and "wm" tools when run from adb shell due to exec spawning #190

Closed
smaeul opened this Issue Feb 24, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Feb 24, 2016

Contributor

I've seen that crash before. It's one of the Android bugs uncovered by using exec spawning. I expect you would hit the same issue if you enabled Android's wrap feature for those processes on stock (not available outside of userdebug/eng builds though).

Contributor

thestinger commented Feb 24, 2016

I've seen that crash before. It's one of the Android bugs uncovered by using exec spawning. I expect you would hit the same issue if you enabled Android's wrap feature for those processes on stock (not available outside of userdebug/eng builds though).

@thestinger thestinger added this to the Release milestone Mar 28, 2016

@thestinger thestinger changed the title from SIGSEGV in "am" and "wm" tools when run from adb shell to SIGSEGV in "am" and "wm" tools when run from adb shell due to exec spawning Mar 28, 2016

@thestinger thestinger added the Nexus 5 label Jun 4, 2016

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Jun 4, 2016

Contributor

This only happens on the Nexus 5, as it appears to be caused by PaX changes that are still active when soft mode is enabled. It's probably the changes related to the stack gap.

If it was only relevant to the Nexus 5, I would just close this, but ART should be improved to work with PaX because it's going to be relevant in the future.

Contributor

thestinger commented Jun 4, 2016

This only happens on the Nexus 5, as it appears to be caused by PaX changes that are still active when soft mode is enabled. It's probably the changes related to the stack gap.

If it was only relevant to the Nexus 5, I would just close this, but ART should be improved to work with PaX because it's going to be relevant in the future.

@thestinger

This comment has been minimized.

Show comment Hide comment
@thestinger

thestinger Jun 5, 2016

Contributor

This is fixed.

Contributor

thestinger commented Jun 5, 2016

This is fixed.

@thestinger thestinger closed this Jun 5, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment