nutomic/syncthing-android
1
0
Fork 0
mirror of https://github.com/syncthing/syncthing-android.git synced 2024-12-27 21:31:29 +00:00
Code Releases Activity

Fix sending wrongly encoded commands into the unix subshell (fixes #1198)

Browse source
This commit is contained in:
Catfriend1 2018-07-28 00:34:55 +02:00
parent 6a459311f4
commit 3967e863f5
1 changed files with 6 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
app/src/main/java/com/nutomic/syncthingandroid/util/Util.java
View file

@ -15,9 +15,11 @@ import android.widget.Toast;
import com.nutomic.syncthingandroid.R;
import com.nutomic.syncthingandroid.service.Constants;
import java.io.BufferedWriter;
import java.io.DataOutputStream;
import java.io.IOException;
import java.io.File;
import java.io.IOException;
import java.io.OutputStreamWriter;
import java.text.DecimalFormat;
import eu.chainfire.libsuperuser.Shell;
@ -139,13 +141,6 @@ public class Util {
useRoot = true;
}
// Check for cyrillic characters in the folder path.
if (absoluteFolderPath.matches(".*\\p{InCyrillic}.*")) {
Log.w(TAG, "nativeBinaryCanWriteToPath: Path '" + absoluteFolderPath +
"' contains cyrillic characters. As this is currently not handled, we'll return true without checking permissions.");
return true;
}
// Write permission test file.
String touchFile = absoluteFolderPath + "/" + TOUCH_FILE_NAME;
int exitCode = runShellCommand("echo \"\" > \"" + touchFile + "\"\n", useRoot);
@ -185,9 +180,10 @@ public class Util {
try {
shellProc = Runtime.getRuntime().exec((useRoot) ? "su" : "sh");
shellOut = new DataOutputStream(shellProc.getOutputStream());
BufferedWriter bufferedWriter = new BufferedWriter(new OutputStreamWriter(shellOut));
Log.d(TAG, "runShellCommand: " + cmd);
shellOut.writeBytes(cmd);
shellOut.flush();
bufferedWriter.write(cmd);
bufferedWriter.flush();
shellOut.close();
shellOut = null;
exitCode = shellProc.waitFor();