Input/Output Probleme beim Ausführen von Shell-Befehlen mit Java

[BaumGuard]

Liebe Java-Community,
ich habe eine grafische Benutzeroberfläche für das Konsolenprogramm chmod geschrieben, allerdings gibt es bei Datei- und Ordnernamen mit Leerzeichen noch Probleme. Für die grafische Oberfläche nutze ich JavaFX.

In meinem Fenster befindet sich ein Button, von dem aus man einen FileChooser öffnen kann. Wenn man nun mit dem FileChooser eine Datei auswählt, wird der Dateipfad im File-Objekt gespeichert. Den Dateipfad speichere ich dann in einer weiteren String. Mit einer if-Abfrage wird geprüft, ob in dem Dateipfad Leerzeichen vorkommen. Wenn das der Fall ist, wird das String mit der Methode split bei den Schrägstrichen in seine Bestandteile (also die Ordner) zerlegt und in einem String-Array gespeichert. Mit der Methode join werden die Array-Elemente in einem weiteren String wieder zusammengefügt und an jedes Element wird vorne ein '/ angehängt. Im nächsten Schritt wird mit der Methode replaceAll das '/ durch '/' ersetzt, ganz hinten ein ' angehängt und in einer weiteren String abgespeichert. Im letzten Schritt wird mit der Methode substring aus der vorherigen String der Teil nach dem ersten ' in einer neuen String gespeichert. Diese wird dann als Dateipfad verwendet.

Nun wird in einer neuen String entsprechend den Nutzereingaben der chmod-Befehl erzeugt und dann als
Process p = Runtime.getRuntime().exec(Befehl);
ausgeführt.

Nun lasse ich mir mit System.out.println noch den Befehl ausgeben, der dem Process übergeben wird.
Wenn ich nach dem Ausführen durch Java mir im Terminal mit ls -l die Zugriffsrechte anzeigen lasse, hat sich allerdings nichts geändert. Kopiere ich mir stattdessen den ausgegebenen Befehl und führe ihn im Terminal aus, funktioniert alles einwandfrei.


Hier ist der Code zur Erzeugung des Dateipfades:

if (dateipfad.contains(" "))
       {
        String [] dateiArray = dateipfad.split("/");
       
        //Dateipfad mit Anführungszeichen bauen
        String zusammen = String.join("/'", dateiArray);
        String zusammen2 = zusammen.replaceAll("/'", "'/'")+ "'";
      
        zusammen3 = zusammen2.substring(1);
        };

Hier ist der Code zur Erzeugung des Befehls:

try  {      
       if (andere.getSource()==OK) {
         
           boolean SchreibenString = SchreibenCB.isSelected();
           boolean LesenString = LesenCB.isSelected();
           boolean AusfuehrenString = AusfuehrenCB.isSelected();
           boolean rekursivString = rekursiv.isSelected();
          
           if (EVString.equals("Entziehen")) {EVParameter="-";};
           if (EVString.equals("Vergeben")) {EVParameter="+";};
          
           if (SchreibenString==true) {
              w="w";  
           } else {w="";};
          
           if (LesenString==true) {
               r="r";
           } else {r="";};
           if (AusfuehrenString==true) {
               x="x";
           } else {x="";};
     
           if (BenutzerString.equals("Ihnen")) {BenutzerParameter="u";};
           if (BenutzerString.equals("Allen außer Ihnen")) {BenutzerParameter="o";};
           if (BenutzerString.equals("Gruppe")) {BenutzerParameter="g";};
           if (BenutzerString.equals("Allen")) {BenutzerParameter="a";};
          
           if (rekursivString==true) {rekursivParameter= "-r";}
           else {rekursivParameter="";};
          
           //Erzeugen und Ausgabe des Befehls
           String Befehl = ("chmod " + rekursivParameter  + " " + BenutzerParameter + EVParameter + w+r+x + " " + zusammen3);
         
       System.out.println(Befehl);
          
       //Ausführen des Befehls.
      
               try {
                   Process p = Runtime.getRuntime().exec(Befehl);
                  
               } catch (IOException e) {
                   // TODO Auto-generated catch block
                   Fehler.setText("Fehler");
               }
          
       }}
       catch (java.lang.NullPointerException f) {Fehler.setText("Es wurden noch nicht alle Daten eingegeben.");}
   }

Wo liegt das Problem?

Für eure Hilfe danke ich euch im Voraus.

 

[mihe7]

Wo liegt das Problem?

Dass Du keine Code-Tags verwendet hast: [CODE=Java]Dein Code[/CODE]

 

[mihe7]

Zur Frage:

1. Du machst das unnötig kompliziert.
2. Würde ich meinen, dass Du den Pfad zu chmod oder zu sh brauchst

Probier mal:

String[] arr = {"/bin/chmod", "777", "/path/to/file"};
Runtime.exec(arr);

 

[BaumGuard]

Muss dann nach dem /bin/chmod noch chmod stehen, sodass das etwa so aussieht?:

/bin/chmod chmod u-r /path/to/file

 

[mihe7]

Nein. Du übergibst einfach jeden Teil der Befehlszeile als String in einem Array.

 

[mihe7]

Habe gerade gemerkt, dass der PATH anscheinend doch durchsucht wird:

public class Test {
    public static void main(String[] args) throws Exception {
        Runtime.getRuntime().exec(args);
    }
}

Dann

javac Test.java
ls -l
-rw-r--r-- 1 xxxx xxxx 443 Jan  5 22:28 Test.class
-rw-r--r-- 1 xxxx xxxx 131 Jan  5 22:27 Test.java

java -cp . Test chmod 777 Test.class
ls -l
-rwxrwxrwx 1 xxxx xxxx 443 Jan  5 22:28 Test.class
-rw-r--r-- 1 xxxx xxxx 131 Jan  5 22:27 Test.java

 

[mihe7]

Bzgl. unnötig kompliziert meinte ich übrigens, dass Du den String nicht extra behandeln musst, wenn du es als Array übergibst:

public class Test {
    public static void main(String[] args) throws Exception {
        Runtime.getRuntime().exec(new String[]{"chmod", "777", "X Y"});
    }
}

Dann

javac Test.java
touch "X Y"
ls -l
java -cp . Test
ls -l

 

[BaumGuard]

Dass ich das vielleicht etwas komplizierter mache, als es eigentlich geht, ist ja hier eigentlich nicht das Problem. Das Problem scheint die korrekte Übernahme des Strings in die Runtime zu sein. Ich habe jetzt das einfache chmod durch /bin/chmod ersetzt. Es funktioniert aber immer noch nicht. Wenn ich den Befehl aber im Terminal ausführe, dann funktioniert es einwandfrei.

Der Befehl hat jetzt folgende Form:
/bin/chmod a+w /'datei'/'pfad'

 

[mihe7]

Das Problem scheint die korrekte Übernahme des Strings in die Runtime zu sein.

Du verstehst nicht ganz: es gibt keinen Befehl "chmod xyz abc". Es gibt einen Befehl "chmod" (oder eben "/bin/chmod").

Wenn Du das Testprogramm aus #6 nimmst, und mit dem folgendes ausführst:

java -cp . Test "chmod 777 Test.class"

dann wirst Du sehen, was ich meine

 

[BaumGuard]

Steht "Test" für den Namen der Hauptklasse?

 

[mihe7]

Klar, in dem Code oben wird eine öffentliche Klasse namens Test definiert. Diese Klasse enthält die main-Methode.

 

[BaumGuard]

Dann muss das Programm wahrscheinlich erst kompiliert werden, bevor man den Befehl aus #9 nutzen kann, oder?
Wenn ja, als jar oder class?

 

[mihe7]

Nimm einfach einen Texteditor, kopier den Code rein, speichere die Datei als Test.java ab. Dann öffnest Du ein Terminal, wechselst in das Verzeichnis und führst die in Kommentar #6 genannten Schritte aus.

 

[BaumGuard]

Funktioniert immer noch nicht. Ich lade jetzt einfach den Code hoch, sodass du selbst mal danach schauen kannst. Die Datei darf auch verändert werden. Die Datei muss dann noch in chmodGUI.java umbenannt werden, weil ich sie nicht als java-Datei hochladen kann.

 

[mrBrown]

Wie schon von @mihe7 gesagt:

Bzgl. unnötig kompliziert meinte ich übrigens, dass Du den String nicht extra behandeln musst, wenn du es als Array übergibst:

Du musst da keinerlei Strings selbst zusammenbasteln oder escapen.
Übergib einfach die Werte alle einzeln (Programmname, Parameter (-r, u+x, etc...), DateiPfad).

 

[BaumGuard]

Vielleicht hätte ich folgende Information noch ergänzen sollen: Wenn ich von meinem Java-Programm aus chmod auf einen Dateipfad, der keine Leerzeichen enthält, anwende, dann funktioniert es problemlos. Nur bei Dateifaden mit Leerzeichen geht es nicht. Dann kann ich die Parameter meinetwegen auch als Arrays der Runtime übergeben, ohne dass das weiterhilft.
Ist es auch möglich, dass es sich um einen Bug in Java handelt?

 

[mrBrown]

Vielleicht hätte ich folgende Information noch ergänzen sollen: Wenn ich von meinem Java-Programm aus chmod auf einen Dateipfad, der keine Leerzeichen enthält, anwende, dann funktioniert es problemlos. Nur bei Dateifaden mit Leerzeichen geht es nicht. Dann kann ich die Parameter meinetwegen auch als Arrays der Runtime übergeben, ohne dass das weiterhilft.
Ist es auch möglich, dass es sich um einen Bug in Java handelt?

Nein, auch Pfade mit Leerzeichen funktionieren mit exec völlig Problemlos.
Dein Versuch, die Leerzeichen zu escapen und den String zusammen zu basteln, macht aber vermutlich mehr kaputt machen als es hilft...

 

[mihe7]

Ist es auch möglich, dass es sich um einen Bug in Java handelt?

Möglich ist alles, aber unwahrscheinlich. Nimm Dir doch einfach mal irgendein konkretes Beispiel auf Deiner Platte - inkl. Leerzeichen (sagen wir mal /home/BaumGuard/Meine Bilder/x.jpg). Dann nimmst Du das simple Testprogramm aus #7, ersetzt "X Y" durch "/home/BaumGuard/Meine Bilder/x.jpg" und lässt es laufen.

 

[BaumGuard]

Nimm Dir doch einfach mal irgendein konkretes Beispiel auf Deiner Platte - inkl. Leerzeichen (sagen wir mal /home/BaumGuard/Meine Bilder/x.jpg).

Genau das mache ich ja schon die ganze Zeit.

Ich habe meine Parameter jetzt in den folgenden Array gepackt:

String [] BefehlArray = new String[] {"/bin/chmod",rekursivParameter,BenutzerParameter,EVParameter,w,r,x,zusammen3};

Dann habe ich der Runtime den Array übergeben:

Process p = Runtime.getRuntime().exec(BefehlArray);

Aber es funktioniert immer noch nicht.

 

[mrBrown]

Übergib einfach den ganz normalen Dateipfad, ohne irgendetwas an diesem zu verändern.

Und natürlich müssen die einzelnen übergebenen Dinge sinnvoll sein, zB einfach leere Strings übergeben ist das nicht.

 

[mihe7]

Genau das mache ich ja schon die ganze Zeit.

Wenn Du das gemacht hättest, dann hättest Du jetzt auch irgendein Ergebnis. Mit dem Aufruf in Deinem Code kommst Du nicht weit. Aktuell führst Du
"/bin/chmod" "-r" "r" "w" "x" gefolgt von Deinem verkorksten Pfad aus.

 

[httpdigest]

Ich habe meine Parameter jetzt in den folgenden Array gepackt:

String [] BefehlArray = new String[] {"/bin/chmod",rekursivParameter,BenutzerParameter,EVParameter,w,r,x,zusammen3};

Der "mode" Parameter des chmod Aufrufes darf nur ein einziger Shell-Parameter sein. Das heißt, hier musst du BenutzerParameter, EVParameter, w, r und x tatsächlich zusammenkonkatenieren und nicht als mehrere String-Array-Elemente setzen.
Siehe: https://en.wikipedia.org/wiki/Chmod#Command_syntax

 

[BaumGuard]

Übergib einfach den ganz normalen Dateipfad, ohne irgendetwas an diesem zu verändern.

Hab ich gemacht.

Der "mode" Parameter des chmod Aufrufes darf nur ein einziger Shell-Parameter sein. Das heißt, hier musst du BenutzerParameter, EVParameter, w, r und x tatsächlich zusammenkonkatenieren und nicht als mehrere String-Array-Elemente setzen.

Hab ich gemacht.

String [] BefehlArray = new String[] {"/bin/chmod",rekursivParameter,g,dateipfad};

Das g ist der von httpdigest erwähnte mode-Parameter. Wenn ich mir diesen ausgeben lasse, ist er richtig.

Es ist noch keine Besserung eingetreten.

 

[httpdigest]

Dann lass uns doch mal debuggen, was chmod eventuell ausgibt:

String[] cmd = {"/bin/chmod",rekursivParameter,g,dateipfad};
new ProcessBuilder(cmd).inheritIO().start().waitFor();

 

[BaumGuard]

Dann lass uns doch mal debuggen, was chmod eventuell ausgibt:

Da wird das ausgegeben:
/bin/chmod: Ungültiger Modus: „“
„/bin/chmod --help“ liefert weitere Informationen.

 

[httpdigest]

Und jetzt debugge mal, was `g` genau enthält.

 

[BaumGuard]

Ich habe mir den Inhalt von g anzeigen lassen und da kommt zum Beispiel so etwas raus wie a-w. Sollte also korrekt sein, oder?
Eben habe ich auch meinen zusammengebauten Befehl-String eingesetzt und es gab eine java.io.IOException. Außerdem schreibt er "Datei oder Verzeichnis nicht gefunden". Wenn ich den Befehl aus dem Stacktrace im Terminal ausführe, funktioniert es ohne Probleme.

 

[mihe7]

Eben habe ich auch meinen zusammengebauten Befehl-String eingesetzt und es gab eine java.io.IOException.

Aha, knapp einen Tag später...

Ich habe mir den Inhalt von g anzeigen lassen und da kommt zum Beispiel so etwas raus wie a-w.

Quizzfrage: welcher der beiden anderen Strings könnte denn noch leer sein?

 

[BaumGuard]

So, jetzt funktioniert es. Ich habe gerade festgestellt, dass ich aus Versehen den Rekursiv-Parameter kleingeschrieben habe, der eigentlich groß sein sollte. Manchmal sieht man eben den Wald vor lauter Bäumen nicht.
Trotzdem vielen Dank für eure Hilfe und Bemühungen!