execl mehrere Parameter übergeben

supertux

einfacher geht es so:

char params[] = "vlc -f --play-and-exit --key-quit=esc --volume=256";

...

execlp("sh", "sh", "-c", params, NULL);

rüdiger

supertux schrieb:

einfacher geht es so:

char params[] = "vlc -f --play-and-exit --key-quit=esc --volume=256";

...

execlp("sh", "sh", "-c", params, NULL);

Das hat er ja schon. Aber will es nicht so.

_frank_

der andere Weg mit der Umwandlung in ein char*-Array hat den nachteil, dass er Probleme mit Leerzeichen hat, da er die als Trennzeichen (Ersetzung durch \0) nimmt.
Ich möchte dem Player ja eine Datei mitgeben, und diese kann Leerzeichen im Pfad enthalten.

hat da vllt. auch jemand eine Idee?

ich bin mit der speicher-allokierung nicht ganz so vertraut. evtl. könnte ich das array eins (1*sizeof(*args)) größer machen und ins letzte Element einen Pointer auf das Datei-Char* (bzw. string.c_str()) reinschreiben...

vorher müsste ich aber meinen command noch vorne an meine Parameter ranbasteln
mit der Konvertierung nach String ist das aber kein Problem denke ich

Als Hintergrundinfo...meine Ursprungsfunktion sieht so aus:

void RunViewer(const char *player, const char *options, const char *file)
{
  string command=string(player)+" "+string(options)+" \""+string(file)+"\""; 
  if (FileExists(file))
  {
    Last_Viewed.SetValue(file,GetTimeString());
    system(command.c_str());
  } else
    g_print("error opening file...\n");
}

ich möchte nur das System ersetzen um den Player in den Hintergrund zu schieben...

supertux

rüdiger schrieb:

supertux schrieb:
einfacher geht es so:
char params[] = "vlc -f --play-and-exit --key-quit=esc --volume=256";

...

execlp("sh", "sh", "-c", params, NULL);
Das hat er ja schon. Aber will es nicht so.

Ohh, das habe ich glatt übersehen. Wie dem auch sie, wieso einfacher, wenn es doch immer schwerer geht

rüdiger

frank schrieb:

ich bin mit der speicher-allokierung nicht ganz so vertraut. evtl. könnte ich das array eins (1*sizeof(*args)) größer machen und ins letzte Element einen Pointer auf das Datei-Char* (bzw. string.c_str()) reinschreiben...

Ja natürlich. Du könntest den Rest der Optionen idealerweise auch gleich in einem besseren Format speichern.

_frank_

rüdiger schrieb:

Du könntest den Rest der Optionen idealerweise auch gleich in einem besseren Format speichern.

wie meinst du das? will die Funktion selbst auf std::string umbauen (nach außen)

dann müsste ich das eigentlich einmal (mit arg0 zusammen) in ein char[] kopieren, oder? sonst klappt das ja nicht mit dem Umbau auf das Array

rüdiger

Naja bekommst du die Optionen und den Dateinamen vom User oder sind die Optionen fix bzw. von deinem Programm generiert? Wenn sie von deinem Programm sind, dann könntest du sie natürlich gleich so speichern, dass du sie an execvp weitergeben kannst.

_frank_

das programm und die optionen werden aus einer textdatei ausgelesen und im Programm in einer map<string,string> gespeichert.

es gibt aktuell 4 Paare dieser Konfiguration (programm, optionen), daher wäre es schon sinnvoll das array dynamisch aus dem string zu erzeugen.

der Dateiname kommt vom Benutzer (wird vorher auch als string zusammengebaut).

_frank_

also ich habs jetzt so:

bool Exec(string command,string parameters,string file)
{
  pid_t pid=fork();
  if (pid==0)//child-process
  {
    //char params[] = "vlc -f --play-and-exit --key-quit=esc --volume=256"; // wichtig! Erster Parameter ist argv[0]!
    char params[command.size()+1+parameters.size()+1];
    char file_char[file.size()];

    strcpy(params,command.c_str());
    strcat(params," ");
    strcat(params,parameters.c_str());
    //g_print("Params: %s\n",params);

    strcpy(file_char,file.c_str());

    size_t number_of_params = 1;
    for(char *p = params; *p; ++p) {
      if(*p == ' ') {
        ++number_of_params;
      }
    }
    number_of_params++;//increase Param-count to hold file
    //g_print("Parameter-Anzahl: %d\n",number_of_params);
    char **args = (char**)malloc( (number_of_params+1) * sizeof(*args));
    if(!args) {
      perror("malloc");
      return false;
    }

    size_t i = 1;
    args[0] = params;
    for(char *p = params; *p; ++p) {
      if(*p == ' ') {
        assert(i < number_of_params);
        args[i++] = p+1;
        *p = '\0';
      }
    }
    args[number_of_params-1] = file_char;
    args[number_of_params] = NULL;

    if(execvp(command.c_str(), args) == -1) 
    {
      return false;
    } else return true;
  } else if (pid==-1)
    return false; //problem while forking
  else return true;//parent process
}

funktioniert soweit alles~~, und keine Abstürze ;)~~
denke mal, die speicherallokierung stimmt so, musste nur die strings kopieren, da "string.c_str()" ein "sonst char*" ist und in dem array aber normale char* sein müssen...

EDIT: wenn das programm nicht existiert, stürzt das Programm ab kann mir jemand sagen, warum?
und wie kann man dem vorbeugen?

Error: Wiedergabeprogramm konnte nicht gestartet werden!
mediaselector: Fatal IO error 2 (No such file or directory) on X server :0.0.
mediaselector: ../../src/xcb_io.c:249: process_responses: Assertion `(((long) (dpy->last_request_read) - (long) (dpy->request)) <= 0)' failed.
Aborted

die erste Zeile ist eine ausgabe von mir wo der rückgabewert meiner exec-funktion ausgewertet wird, gleichzeitig versuche ich ein GTK-Control zu modifizieren (Anzeige der gleichen Fehlermeldung in einer InfoBar)...ich denke mal, da knallts.
mit dem Handling

perror("execvp");
      _Exit(1);

wenn die execvp nicht fehlschlägt, stürzt das Programm auch nicht ab.
Aber ich habe keine Rückmeldung, an der ich feststellen kann, ob der prozess korrekt gestartet werden konnte.
Ich gebe doch mit _Exit(...) den errorcode mit, den muss ich doch irgendwie vom parent auslesen können...
wie kann ich dem parent sagen, dass der Child-prozess fehlgeschlagen ist?

Gruß Frank

_frank_

Hi,
hat jemand eine idee (siehe vorheriger Post)?

außerdem hab ich festgestellt, dass nach der benutzung von execvp ein zombo-prozess zurückbleibt. wie kann ich das verhindern? muss ich das fork aufheben?

Gruß Frank