Skip to content
This repository was archived by the owner on Feb 13, 2024. It is now read-only.

disable FLAG_SECURE #7

Open
wants to merge 1 commit into
base: anbox/master
Choose a base branch
from

Conversation

Venji10
Copy link

@Venji10 Venji10 commented May 18, 2021

FLAG_SECURE prevents some applications using it from showing up in anbox.
I think we should disable it for anbox because it just prevents some apps from running
and doesn't offer an actual benefit for us.
Here is a patch for that.

FLAG_SECURE prevents some applications from showing up in anbox.
@Newbytee
Copy link

Commenting out code, especially without an inline explanation as to why it's commented out, is really bad practice. Could you also share some examples of apps where this helps?

@Venji10
Copy link
Author

Venji10 commented May 25, 2021

@Newbytee yeah, maybe I should add an explanation.

I primarily tested it with https://github.com/gwpantazes/Flag-Secure-Sample/releases .
Without this patch the "secure" app doesn't show up, with it it does show up.
For me, my banking app which uses FLAG_SECURE now works with this patch, even though I am not sure if I will actually use this.

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

Successfully merging this pull request may close these issues.

2 participants