Seite 1 von 1
os.system Rückgabewert Problem
Verfasst: Sonntag 25. Mai 2008, 12:30
von Chrispy
Da der Standard GNOME Run Dialog keine Variablen akzeptiert habe ich mich entschlossen einfach selber einen in PyGTK zu schreiben, bis auf den Rückgabewert von os.system klappt das ja auch wunderbar.
Den Wert hole ich mir wie folgt:
self.execute = os.system("was weiss ich")
self.signal = self.execute & 0xFF
self.exitcode = (self.signal >> 8 ) & 0xFF
Direkt im Python Interpreter _kein_ Problem, aber im Script funktioniert das nicht...
Hier noch die ganze Funktion zur Programmausführung ...
Code: Alles auswählen
def run(self, command):
self.infolabel.set_text("")
if self.terminal_checkbox.get_active() and self.root_checkbox.get_active():
self.command = "gksu -u root \"%s -e `bash -c \'%s\'`\"" % (self.terminal, self.entry_command.get_text())
self.execute = os.system("%s &" % self.command)
elif self.terminal_checkbox.get_active() and self.root_checkbox.get_active() != True:
self.command = "%s -e \"bash -c \'%s\'\"" % (self.terminal, self.entry_command.get_text())
self.execute = os.system("%s &" % self.command)
elif self.terminal_checkbox.get_active() != True and self.root_checkbox.get_active():
self.command = "gksu -u root %s" % self.entry_command.get_text()
self.execute = os.system('%s &' % self.command)
else:
self.command = "%s" % (self.entry_command.get_text())
self.execute = os.system("%s &" % self.command)
self.signal = self.execute & 0xFF
self.exitcode = (self.signal >> 8) & 0xFF
print "%s" % self.exitcode
if self.exitcode == 127:
self.infolabel.set_text(_("Command not found"))
elif self.exitcode != 0:
print "Something went wrong"
self.infolabel.set_text(_("Something went wrong"))
else:
if self.command not in self.history:
self.history.append(self.command)
self.history_list.append_text(self.command)
gconf_database.set_list("/apps/gnome-settings/gnome-panel/history-gnome-run", "string", self.history)
gtk.main_quit()
Ich hoffe jemand hat ne Antwort parat.
Verfasst: Sonntag 25. Mai 2008, 12:45
von lunar
Anstatt magisch Bytes rumzuschieben, könntest du auch einfach ``os.WEXITSTATUS``, ``os.WTERMSIG`` und ``os.WIFSIGNALED`` nutzen.
Ferner solltest du dich fragen, ob die Shell Injection wirklich beabsichtigt ist, oder ob du zumindest deine Quoting-Logik nicht dahingehen verbessern möchtest, dass Werte mit Hochkommata korrekt gequotet werden.
Zuguterletzt wäre eine etwas ausführlichere Fehlerbeschreibung als "funktioniert nicht" hilfreich
Verfasst: Sonntag 25. Mai 2008, 12:49
von Chrispy
Zuguterletzt wäre eine etwas ausführlichere Fehlerbeschreibung als "funktioniert nicht" hilfreich Wink
Äh ja. Das Ergebnis ist im Interpreter immer korrekt (0 oder 1 oder 127 usw.) im Script ist es immer 0 (auch dann wenn es 1 oder 127 wäre)
oder ob du zumindest deine Quoting-Logik nicht dahingehen verbessern möchtest, dass Werte mit Hochkommata korrekt gequotet werden.
Gut, aber wie? Python will die Dinge Gequotet haben, gnome-terminal und gksu auch nochmal extra. Dies hier war die bis dato einzige funktionierende Lösung es allen dreien Recht zu machen.
Verfasst: Sonntag 25. Mai 2008, 13:05
von Leonidas
Man könnte ja subprocess.Popen nutzen, dann hat sich das Quoten auch erledigt. Ich denke lunar wollte auf genau das anspielen.
Verfasst: Sonntag 25. Mai 2008, 13:33
von Chrispy
Hmm. Zumindest das mit dem Rückgabewert funktioniert jetzt einwandfrei. :freu:
Mal sehen ob ich das mit gksu und gnome-terminal auch noch hinbekomme.
Verfasst: Sonntag 25. Mai 2008, 13:41
von lunar
Chrispy hat geschrieben:Gut, aber wie?
Tja, das ist eine gute Frage, die schon so Manchem Kopfschmerzen bereitet hat. Deswegen vermeidet man ja die Shell auch soweit als möglich, und nutzt lieber ``subprocess.Popen``, sofern man nicht unbedingt auf die Shell angewiesen ist.
Dein Problem ist, dass ``os.system`` immer den Exitcode von ``/bin/sh`` zurückgibt. Für den Fall, dass das aufgerufene Skript in der Shell terminiert, gibt ``/bin/sh`` den Exitcode des letzten Kommandos zurück.
Du allerdings schiebst den Prozess mit ``&`` in den Hintergrund. Das führt dazu, dass die Shell eine neue Subshell forkt, die SIGHUP ignoriert, und das Kommando ausführt. Danach beendet sich die Shell. Die geforkte Subshell lebt zu diesem Zeitpunkt noch, ergo kann die Shell deren Rückgabewert nicht abfragen. Folglich gibt sie ihren eigenen zurück, und da beim Forken der Subshell nun mal kein Fehler auftrat, ist der Statuscode immer 0.
Ähnliches gilt wohl bei der Ausführung über ``gksu`` und ``gnome-terminal``.
Gratulation, du gehörst zur langen Reihe von Programmierern, die über Tücken von ``os.system`` und ``/bin/sh`` gestolpert sind
Verfasst: Sonntag 25. Mai 2008, 13:47
von Chrispy
Gratulation
Danke. Danke. Danke.
Ich hab zu gksu + subprocess noch ne Frage
self.command ist in dem Fall:
Code: Alles auswählen
self.execute = subprocess.call(["gksu -u root" + self.command], shell=True)
das shell equivalent wäre dieses:
Hier liegt mein Problem: es müsste eigentlich:
sein, da --no-desktop sonst von gksu als eigene option interpretiert wird ... Rat?
Verfasst: Sonntag 25. Mai 2008, 13:51
von Chrispy
ok. hat sich erledigt.
Ich würd halt mal:
Code: Alles auswählen
self.execute = subprocess.call(["gksu -u root" + "\"" + self.command + "\"" ], shell=True)
Manchmal sind die Knoten im Hirn wirklich groß ...
Verfasst: Sonntag 25. Mai 2008, 13:58
von lunar
Dir ist doch klar, dass du so zwar das Problem der Rückgabe gelöst hast, aber noch nicht das Problem des Quotings?
Verfasst: Sonntag 25. Mai 2008, 14:01
von Chrispy
Richtig. Aber wie gesagt: Ich habe keine Ahnung. Und es ist das erste mal, dass ich subprocess benutze.
Verfasst: Sonntag 25. Mai 2008, 14:08
von lunar
Wie wäre es mit
Code: Alles auswählen
self.process = subprocess.call(['gksu', '-u', 'root', self.command])
Verfasst: Sonntag 25. Mai 2008, 14:14
von Chrispy
perfekt.
Noch ne Frage: Eigentlich sollte ein Run Dialog sich ja schliessen, wenn das Programm erfolgreich gestartet wurde, tut er aber nicht. ( dafür war das & am ende des kommandos gedacht, dieses verhindert aber die korrekte Rückgabewert Angabe).