Skip to content

Commit 974e695

Browse files
committed
Temporarily disable the credential check
Some devices are failing this check because the creds are coming back as 0/<superuser>. This needs to be investigated further.
1 parent 22310bf commit 974e695

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/com/noshufou/android/su/SuRequestActivity.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,10 +131,10 @@ public void onCreate(Bundle savedInstanceState) {
131131
Log.e(TAG, "Divided by zero...");
132132
return;
133133
}
134-
if ((creds.getUid() != appInfo.uid || creds.getGid() != appInfo.uid) &&
135-
(creds.getUid() != 0 || creds.getGid() != 0)) {
136-
throw new SecurityException("Potential forged socket, socket uid=" + creds.getUid() + ", gid=" + creds.getGid());
137-
}
134+
// if ((creds.getUid() != appInfo.uid || creds.getGid() != appInfo.uid) &&
135+
// (creds.getUid() != 0 || creds.getGid() != 0)) {
136+
// throw new SecurityException("Potential forged socket, socket uid=" + creds.getUid() + ", gid=" + creds.getGid());
137+
// }
138138
readRequestDetails(suVersionCode, intent);
139139
} else {
140140
Log.w(TAG, "Recieved null socket path, aborting");

0 commit comments

Comments
 (0)