ChainsDD / su-binary

su binary for android Superuser
Other
456 stars 322 forks source link

Possible db fix #2

Closed HomerSp closed 13 years ago

HomerSp commented 13 years ago

You know the details already :)

ChainsDD commented 13 years ago

Does the effective UID/GID not need to be set back to 0 before allowing the app that has requested su?

HomerSp commented 13 years ago

It is already, in allow(): setresgid(to->uid, to->uid, to->uid); setresuid(to->uid, to->uid, to->uid);

ChainsDD commented 13 years ago

Ah, so it is. Merging