CreateProcess() schlägt mit einer Zugriffsverletzung fehl

Question

Mein Ziel ist es, eine externe ausführbare Datei in meinem Programm auszuführen. Zuerst habe ich system() Funktion verwendet, aber ich möchte nicht die Konsole für den Benutzer angezeigt werden. Also, ich suchte ein wenig und fand CreateProcess() Funktion. Wenn ich jedoch versuche, einen Parameter zu übergeben, weiß ich nicht warum, es schlägt fehl. Ich habe diesen Code von MSDN, und ein wenig verändert:

#include <windows.h> 
#include <stdio.h> 
#include <tchar.h> 

void _tmain(int argc, TCHAR *argv[]) 
{ 
    STARTUPINFO si; 
    PROCESS_INFORMATION pi; 

    ZeroMemory(&si, sizeof(si)); 
    si.cb = sizeof(si); 
    ZeroMemory(&pi, sizeof(pi)); 
    /* 
    if(argc != 2) 
    { 
     printf("Usage: %s [cmdline]\n", argv[0]); 
     return; 
    } 
    */ 
    // Start the child process. 
    if(!CreateProcess(NULL, // No module name (use command line) 
     L"c:\\users\\e\\desktop\\mspaint.exe",  // Command line 
     NULL,   // Process handle not inheritable 
     NULL,   // Thread handle not inheritable 
     FALSE,   // Set handle inheritance to FALSE 
     0,    // No creation flags 
     NULL,   // Use parent's environment block 
     NULL,   // Use parent's starting directory 
     &si,   // Pointer to STARTUPINFO structure 
     &pi)   // Pointer to PROCESS_INFORMATION structure 
    ) 
    { 
     printf("CreateProcess failed (%d).\n", GetLastError()); 
     return; 
    } 

    // Wait until child process exits. 
    WaitForSingleObject(pi.hProcess, INFINITE); 

    // Close process and thread handles. 
    CloseHandle(pi.hProcess); 
    CloseHandle(pi.hThread); 
} 

jedoch dieser Code irgendwie Zugriffsverletzung crated. Kann ich mspaint ausführen, ohne dem Benutzer die Konsole anzuzeigen?

Vielen Dank.

Answer 1

Versuchen Sie es, es sollte funktionieren.

TCHAR lpszClientPath[500]= TEXT("c:\\users\\e\\desktop\\mspaint.exe"); 
if(!CreateProcess(NULL, lpszClientPath, NULL, NULL, FALSE, NORMAL_PRIORITY_CLASS|CREATE_NEW_CONSOLE|CREATE_UNICODE_ENVIRONMENT,NULL, NULL, &si, &pi)) 
      { 
    printf("CreateProcess failed (%d).\n", GetLastError()); 
     return; 
      } 
... 
... 

Answer 2

Das zweite Argument ist ein LPTSTR, nämlich ein Zeiger auf ein nicht-const Char-Array. Die docs ausdrücklich sagen:

diese Parameter nicht ein Zeiger auf Nur-Lese-Speicher (wie ein const Variable oder eine Zeichenkette) sein können

Der Grund Führen einen Stringliteral ist ein Problem:

Das System fügt der Befehlszeile ein abschließendes Nullzeichen hinzu, um den Dateinamen von den Argumenten zu trennen. Dies teilt die ursprüngliche Zeichenfolge in zwei Zeichenfolgen für die interne Verarbeitung.

Das bedeutet in Ihrem Fall, es versucht, nur Lese-Speicher zu modifizieren, daher der Absturz.

Answer 3

ändern Sie den Code zu diesem:

#include <windows.h> 
#include <stdio.h> 
#include <tchar.h> 

void _tmain(int argc, TCHAR *argv[]) 
{ 
    TCHAR ProcessName[256]; 
    STARTUPINFO si; 
    PROCESS_INFORMATION pi; 

    wcscpy(ProcessName,L"c:\\users\\e\\desktop\\mspaint.exe"); 
    ZeroMemory(&si, sizeof(si)); 
    si.cb = sizeof(si); 
    ZeroMemory(&pi, sizeof(pi)); 
    /* 
    if(argc != 2) 
    { 
     printf("Usage: %s [cmdline]\n", argv[0]); 
     return; 
    } 
    */ 
    // Start the child process. 
    if(!CreateProcess(NULL, // No module name (use command line) 
     ProcessName,  // Command line 
     NULL,   // Process handle not inheritable 
     NULL,   // Thread handle not inheritable 
     FALSE,   // Set handle inheritance to FALSE 
     0,    // No creation flags 
     NULL,   // Use parent's environment block 
     NULL,   // Use parent's starting directory 
     &si,   // Pointer to STARTUPINFO structure 
     &pi)   // Pointer to PROCESS_INFORMATION structure 
    ) 
    { 
     printf("CreateProcess failed (%d).\n", GetLastError()); 
     return; 
    } 

    // Wait until child process exits. 
    WaitForSingleObject(pi.hProcess, INFINITE); 

    // Close process and thread handles. 
    CloseHandle(pi.hProcess); 
    CloseHandle(pi.hThread); 
}