Exitcodes spawn(), system() [Win32, c]

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Servus,

ich möchte von meinem Programm aus ein anderes (in diesem Fall
7z.exe) aufrufen und benötige dessen exit-Code und das, was es
auf die console ausgibt.

Mit

system( "7x e gaga.zip >datei")

erhalte ich das Ergebnis (zB die Dateiliste) in datei. Ok.
system gibt mir aber nicht 7z's exitcode zurück sondern den des
Kommandoprozessors. Klar kann ich diese Datei auswerten, ist aber
nicht sehr elegant und fehlerträchtug (zB Sprachwechsel).

Mit

spawnv( P_WAIT, "7z", ...parameterliste...);

bekomme ich den Exitcode von 7z aber nicht dessen console-output.
Eine Umleitung der Ausgabe in eine Datei geht dort nicht, das
macht ja der Kommandoprozessor ....

Gibt es da noch einen anderen Weg ?

[Peter II (Gast)]

Joachim Drechsel schrieb:
> Gibt es da noch einen anderen Weg ?

mit createprocess arbeiten und die ausgabe selber in eine Datei 
umleiten.

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Merci. Ich stöbere mal ...

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Servus Peter,

es hat jetzt ein wenig gedauert bis es funktionierte. Ich habe mir
dazu das halbe Web durchwühlt und das was unten steht blieb übrig ;)

cmd ist die auszuführende Kommandozeile.

Unklar ist mir noch ob und welche Handles ich closen muß.
Werden mit CreatePipe() noch andere Sachen geöffnet ?
Muß ich die Pipe auch irgendwie schließen ?

#define BUFSIZE 4096

HANDLE rPipe = NULL;
HANDLE wPipe = NULL;
char   chBuf[BUFSIZE];

        int
        JExecute(
        char    *cmd
        )
{
        SECURITY_ATTRIBUTES saAttr;
        PROCESS_INFORMATION piProcInfo;
        STARTUPINFO         siStartInfo;
        BOOL    bSuccess;
        int     code = 255;

        fprintf( fprt, "\n        executing %s", cmd);
        *chBuf = 0;

        // Set the bInheritHandle flag so pipe handles are inherited
        saAttr.nLength              = sizeof(SECURITY_ATTRIBUTES);
        saAttr.bInheritHandle       = TRUE;
        saAttr.lpSecurityDescriptor = NULL;

        // Create a pipe for the child process's STDOUT
        if ( ! CreatePipe(&rPipe, &wPipe, &saAttr, 0))
                ExitError( "uimpxml", "CreatePipe() - pipe for child process\'s STDOUT failed");

        // Ensure the read handle to the pipe for STDOUT is not inherited
        if ( ! SetHandleInformation(rPipe, HANDLE_FLAG_INHERIT, 0))
                ExitError( "uimpxml", "SetHandleInformation() - pipe STDOUT read handle failed for inheritance");

        ZeroMemory(&piProcInfo, sizeof(PROCESS_INFORMATION));

        ZeroMemory( &siStartInfo, sizeof( STARTUPINFO));
        siStartInfo.cb         = sizeof( STARTUPINFO);
        siStartInfo.hStdError  = wPipe;
        siStartInfo.hStdOutput = wPipe;
        siStartInfo.hStdInput  = NULL;
        siStartInfo.dwFlags   |= STARTF_USESTDHANDLES;

        // Create the child process
        bSuccess = CreateProcess(
          NULL, // Use szCmdLine
          cmd,  // command line
          NULL, // process security attributes
          NULL, // primary thread security attributes
          TRUE, // handles are inherited
          0,    // creation flags
          NULL, // use parent's environment
          NULL, // use parent's current directory
          &siStartInfo, // STARTUPINFO pointer
          &piProcInfo); // receives PROCESS_INFORMATION

        if ( bSuccess) {
                WaitForSingleObject( piProcInfo.hProcess, INFINITE);
                GetExitCodeProcess( piProcInfo.hProcess, (LPDWORD) &code);
                CloseHandle( piProcInfo.hProcess);
                CloseHandle( piProcInfo.hThread);
        }

        ReadFromPipe();

        CloseHandle( rPipe);
        CloseHandle( wPipe);

        return code;
}

        // Read output from the child process's pipe for STDOUT and
        // write to the parent process's pipe for STDOUT. Stop when
        // there is no more data.

        void
        ReadFromPipe()
{
        DWORD   dwRead;
        BOOL    bSuccess = FALSE;
        int     i;

        CloseHandle( wPipe);

        fprintf( fprt, "\n        > ");
        for( ; ; ) {
                bSuccess = ReadFile( rPipe, chBuf, BUFSIZE, &dwRead, NULL);
                if(!bSuccess || dwRead == 0)
                        break;
                for ( i=0; i<dwRead; i++){
                        switch ( chBuf[i]) {
                          case '\r':
                                break;
                          case '\n':
                                fprintf( fprt, "\n        > ");
                                break;
                          default     : ;
                                fputc( chBuf[i], fprt);
                        }
                }
        }
}

[Rufus Τ. F. (rufus)]

Joachim Drechsel schrieb:
> Muß ich die Pipe auch irgendwie schließen ?

Das machst Du doch:

>   CloseHandle( rPipe);
>   CloseHandle( wPipe);

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Das reicht ? Ich hatte so irgendwie das Gefühl, da sei wieder
irgendwo noch ein kleines Unheil ...

Die Funktion wird uU einige 100mal hintereinander aufgerufen,
nicht, daß ich da irgendwelche Leaks hineinbekomme.

Merci !

[Peter II (Gast)]

Joachim Drechsel schrieb:
> Die Funktion wird uU einige 100mal hintereinander aufgerufen,
> nicht, daß ich da irgendwelche Leaks hineinbekomme.

dann schau doch einfach in den Taskmanager nach. Dort kann man die 
Spalte Handles anzeigen. Wenn der wert nicht steigt ist alles ok.

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Peter II schrieb:
> dann schau doch einfach in den Taskmanager nach. Dort kann man die
> Spalte Handles anzeigen. Wenn der wert nicht steigt ist alles ok.

Bleibt gleich (wenn die Anzeige stimmt) ;)

Ich lasse das mal unter Last laufen. Mal schauen was da heraus kommt. 
:-)

Vielen Dank an euch alle !