CreateProcess Problem

Page 1 of 2

11/28/2009 21:13 MrSm!th#1

PHP Code:


			
STARTUPINFO si;

    PROCESS_INFORMATION pi;

    ZeroMemory( &si, sizeof(si) );

    si.cb = sizeof(si);

    ZeroMemory( &pi, sizeof(pi) );





    TCHAR buf[MAX_PATH];

    GetCurrentDirectory(MAX_PATH,buf);

    TCHAR path[MAX_PATH] = L"\"";

    _tcscat_s(path,buf);

    _tcscat_s(path,_T("\\S4Client.exe"));

    TCHAR param1[] = L" -rc:eu -lac:eng -auth_server_ip:217.110.102.94\"";

    TCHAR param2[] =L" -rc:eu -lac:ger -auth_server_ip:217.110.102.94\"";

    TCHAR path2[MAX_PATH];

    _tcscpy(path2,path);

    _tcscat_s(path,L"\"");



    if(lac == 1)

    {

        _tcscat_s(path2,param1);

        if(!CreateProcess(path,path2,NULL,NULL,FALSE,0,NULL,NULL,&si,&pi))

        {

            CString str;

            str.Format( L"Error: %i", ::GetLastError());

             MessageBox( str, L"Error!", MB_OK );

        }

    }

    else if(lac == 2)

    {        

        _tcscat_s(path2,param2);

        if(!CreateProcess(path,path2,NULL,NULL,FALSE,0,NULL,NULL,&si,&pi))

        {

            CString str;

            str.Format( L"Error: %i", ::GetLastError());

             MessageBox( str, L"Error!", MB_OK );

        }

    }

    else

        AfxMessageBox(_T("Keine Sprache gew�hlt./ No Language selected!"));

AfxMessageBox(path);

AfxMessageBox(path2);

Hab mal den Code zuerst gepostet.
So hi erstmal und zu meinem Problem^^:

Der Folgende Code soll ein kleiner Launcher f�r das Spiel S4-League werden.
Die Variable lac ist die Wahl der Sprache und bekommt mit 2 Radiobuttons ihren Wert.
Der Client erwartet die Parameter:
-rc: (f�r uns nur eu)
-lac: (sprache; in meinem launcher nur eng und ger)
-auth_server_ip: (die ip des loginservers die meines wissens f�r eu gleich ist, wenn nicht, kann ich sie ja noch �ndern; das problem tritt schon vorher auf!)

So ich habe schon etliches Probiert! Seit 16 Uhr bin ich am googlen und umschreiben, aber ich kriegs nicht gebacken.
Mal ist der Fehlercode 123 (hei�t Fehler in der Aufrufsyntax; verwirrt mich ziemlich, denn wenn ich den ersten Parameter von CreateProcess weglasse, kommt 745 (bin mir jetzt nicht mehr sicher), obwohl bei beiden M�glichkeiten unten von den zwei DebugMessageBoxen der selbe Pfad ausgegeben wird!), mal 2 und eben auch mal 745.
Ich bin echt am Verzweifeln und finde einfach nicht die L�sung.
Der Pfad ist immer richtig [habe es einmal mit absolutem Pfad probiert und einmal mit relativem (beim 1. u. 2. Parameter nur S4Client.exe (beim zweiten noch die Parameter)) und auch einmal halb halb (beim 1. relativ beim 2. absolut)]!
Es will einfach nicht klappen.
K�nnte mir einer dabei helfen?

Danke im Voraus Sm!th

11/28/2009 22:20 MoepMeep#2

[Only registered and activated users can see links. Click Here To Register...]

11/28/2009 22:30 flo8464#3

Nicht wirklich viel Lust bei dir Fehler zu suchen, hier mal meine (funktionierende) Implementierung in C++ :

Code:

	ProcessId Process::createProcessAndOpen(const std::wstring& applicationName, const std::wstring& commandlineArguments, bool suspended, DWORD waitingTime)
	{
		STARTUPINFO			siStartupInfo;
		PROCESS_INFORMATION piProcessInfo;

		ZeroMemory(&siStartupInfo, sizeof(siStartupInfo));
		ZeroMemory(&piProcessInfo, sizeof(piProcessInfo));
		siStartupInfo.cb = sizeof(siStartupInfo);

		std::wstring formattedArguments(L"\"" + applicationName + L"\" " + commandlineArguments);	

		int ec;
		if(suspended)
			ec = CreateProcessW(applicationName.c_str(), &formattedArguments[0], NULL, NULL, false, CREATE_SUSPENDED, NULL, NULL, &siStartupInfo, &piProcessInfo);
		else
			ec = CreateProcessW(applicationName.c_str(), &formattedArguments[0], NULL, NULL, false, CREATE_DEFAULT_ERROR_MODE, NULL, NULL, &siStartupInfo, &piProcessInfo);

		if(!ec)
			throw std::runtime_error("Process::createProcessAndOpen Error : CreateProcessW() failed");

		if(waitingTime)
		{
			if(WaitForSingleObject(piProcessInfo.hThread, waitingTime) == WAIT_FAILED)
				throw std::runtime_error("Process::createNewProcess Error : WaitForSingleObject() failed");
		}

		processId_		= piProcessInfo.dwProcessId;
		processHandle_	= piProcessInfo.hProcess;
		threadHandle_	= piProcessInfo.hThread;

		return processId_;
	}

11/28/2009 22:56 wurstbrot123#4

Code:

PROCESS_INFORMATION pi = {0};
STARTUPINFO si = {0};
char path[256];
GetCurrentDirectory(256, path);
strcat_s(path, 256, "\\beispiel.exe parameter");
CreateProcess(NULL, path, NULL, NULL, NULL, NULL, NULL, NULL, &si, &pi);

11/28/2009 23:16 MrSm!th#5

Quote:

Originally Posted by MoepMeep

[Only registered and activated users can see links. Click Here To Register...]

sehr hilfreich
als ob ich da nicht schon gewesen w�re :rolleyes:
danke wurstbrot.
werde mal sehen, dass ich das einbaue

edit:

hm 1. das zweite was du gepostet hast, wird nicht klappen, wegen den parametern (fehlende extra ")
da m�sste man dann noch die parameter dranh�ngen + "" um alles
2. das erste:
eine frage
was bedeutet formattedArguments[0]?
ist das nicht das erste zeichen sprich "? warum genau gibst du das als Argument f�r Cmdline?

11/28/2009 23:32 flo8464#6

Quote:

hm 1. das zweite was du gepostet hast, wird nicht klappen, wegen den parametern (fehlende extra ")
da m�sste man dann noch die parameter dranh�ngen + "" um alles

Nein. Programmname in Anf�hrungszeichen, Parameter einfach anh�ngen ;)

Quote:

2. das erste:
eine frage
was bedeutet formattedArguments[0]?
ist das nicht das erste zeichen sprich "? warum genau gibst du das als Argument f�r Cmdline?

Ein Zeiger auf das erste Zeichen, h�tte auch c_str() aufrufen k�nnen. Hatte fr�her das Programm anders strukturiert und ich hab nen std::vector<wchar_t> �bergeben, das k�nnte ich mal ausbessern obwohl es egal ist.

11/28/2009 23:42 MrSm!th#7

Quote:

Originally Posted by flo8464

Nein. Programmname in Anf�hrungszeichen, Parameter einfach anh�ngen ;)
.

das hatte ich ja vorher
da kam error 123!
im internet habe ich gelesen, dass man f�r mehrere parameter "" braucht, damit es klappt
da aber auch generell f�r die initialisirung "" gebraucht werden braucht man \"

also TCHAR[MAX_PATH] = L"\" name -arg -arg -arg\""
damit herauskommt

"name -arg -arg -arg", was ja gebraucht wird f�r das starten
ok in der komandozeile braucht man es nicht, aber wie gesagt, ganz am anfang, als ich noch gar nichts mit diesen ganzen \" gemacht habe, hatte ich es so, ohne die anf�hrungszeichen gemacht, wie es wurstbrot als zweites gepostet hat! und das hab nen error

11/29/2009 02:58 flo8464#8

Nein, Argumente in Anf�hrungszeichen zu setzen ist einfach falsch.
Du darfst sie auch nicht als Einheit betrachten.

Was arbeitest du eigentlich in deinem C++-Programm mit so vielen Low-Level Strings?

11/29/2009 10:29 MrSm!th#9

wieso low?

komisch, also in viele problemberichten sehe ich, dass es probleme gibt (lol), wenn man mehrere parameter hat, die durch leerzeichen getrennt sind, weil es dann nicht funktioniert.
naja ich probiers nochmal.

n� sag ich doch.
gibt error 740

PHP Code:


			
STARTUPINFO si;
    PROCESS_INFORMATION pi;
    ZeroMemory( &si, sizeof(si) );
    si.cb = sizeof(si);
    ZeroMemory( &pi, sizeof(pi) );


    TCHAR buf[MAX_PATH];
    GetCurrentDirectory(MAX_PATH,buf);
    TCHAR path[MAX_PATH] = L"";
    _tcscat_s(path,buf);
    _tcscat_s(path,_T("\\S4Client.exe"));
    TCHAR param1[] = L" -rc:eu -lac:eng -auth_server_ip:217.110.102.94";
    TCHAR param2[] =L" -rc:eu -lac:ger -auth_server_ip:217.110.102.94";
    TCHAR path2[MAX_PATH];
    _tcscpy(path2,path);
    //_tcscat_s(path,L"\"");

    if(lac == 1)
    {
        _tcscat_s(path2,param1);
        if(!CreateProcess(NULL,path2,NULL,NULL,FALSE,0,NULL,NULL,&si,&pi))
        {
            CString str;
            str.Format( L"Error: %i", ::GetLastError());
             MessageBox( str, L"Error!", MB_OK );
        }
    }
    else if(lac == 2)
    {        
        _tcscat_s(path2,param2);
        if(!CreateProcess(NULL,path2,NULL,NULL,FALSE,0,NULL,NULL,&si,&pi))
        {
            CString str;
            str.Format( L"Error: %i", ::GetLastError());
             MessageBox( str, L"Error!", MB_OK );
        }
    }
    else
        AfxMessageBox(_T("Keine Sprache gew�hlt./ No Language selected!"));
AfxMessageBox(path);
AfxMessageBox(path2);

Die MessageBox gibt nun aus:

"F:\S4League\S4Client.exe -rc:eu -lac:ger -auth_server_ip:217.110.102.94" (aber ohne die ""! vorher hat er aus gegeben ""..."" (ohne die �u�eren ""^^))#

edit: das gibts doch nicht. ich habe jetzt mal die parameter und damit die leerzeichen weggelassen und es geht immer noch nicht!
error 740
mal schnell googlen

p.s. ein paar ausz�ge aus anderen foren:

Quote:

Quote:

Hallo , sorry, dass ich den alten Beitrag wieder ausgrabe, aber ich habe ein �hnliches Problem.
D.h. eigentlich funktioniert alles so wie es soll, nur wenn die �bergabeparameter ein Leerzeichen enthalten m�ste man sie eigentlich in
"" Zeichen setzen. Macht man das, wird aber der Pfad zum Programm nicht mehr erkannt.
D.h. ich kann das Ganze nur nutzen wenn ich �bergabeparameter ohne Leerzeichen verwende. Hat jemand eine Idee wie man das zum Laufen bekommen k�nnte?
Also ich verwende einen Module Name (Pfad zur CMD.EXE) und in der CommandLine (zweiter Parameter) den Pfad zum Programm, oder zur Batch Datei plus der Startparameter.

Quote:

Wenn man Klammern im Pfad hat geht es auch nicht....
"C:/User/New Folder/my.cmd" klappt
"C:/User/New Folder(2)/my.cmd" klappt nicht.

edit: ahhh 740 ist ein Zugriffsproblem
Launcher mit Adminrechten ausgef�hrt und es ging.
Komisch nur, dass diese zitierten anderen Probleme mit Leerzeichen haben.. =/
Naja danke auf jeden Fall euch beiden.

edit: noch eine Frage:

Wenn du sagst, diese extra " sind falsch, weil man das nicht darf, warum funktioniert dann wurstbrots Implementierung?
Da sind doch auch L"\"" bei!
ah schon gut...^^ bei Leerzeichen im Pfad, m�ssen "" nicht bei den Parametern
Ok, alle Fragen gekl�rt, Launcher fertig und dank wurstbrots Post auch noch eine kleine Extrafunktion eingebaut.

Danke euch beiden^^

11/29/2009 11:51 flo8464#10

Sehr sch�n. ;)

Mit Low-LevelStrings meine ich deine char-Arrays (TCHAR[]).
C++-Strings, also die Strings aus der Standardbibliothek, sind leichter zu bedienen, sicherer und es ist einfach besserer Stil.

Wenn du weiterhin sowohl ASCII als auch Unicodestrings unterst�tzten m�chtest, was ich nichtmal tun w�rde da ASCII auf Windows eigentlich nichts mehr verloren hat, typedef dir nen Stringtyp zusammen. "typedef basic_string<TCHAR> tstring" oder soetwas. ;>

11/29/2009 11:58 MrSm!th#11

Nein ich bin da kein Fan von.
Ich mag CString und std::string am liebsten, aber f�r GetCurrentDirectory hatte ich nunmal nur ein Beispiel mit TCHAR zur Hand und bei CreateProcess konnte als zweiter Parameter CString nicht in LPWSTR konvertiert werden....... und da hatte ich bei so einem kleinen Projekt keine Lust auf gro�artige Konvertierung.
Normalerweise, wie gesagt nutze ich CString und manchmal auch std::string^^

p.s. Zum Teil auch deswegen, weil ich Arrays noch nie mochte xD

11/29/2009 12:23 flo8464#12

Quote:

Originally Posted by MrSm!th

Nein ich bin da kein Fan von.
Ich mag CString und std::string am liebsten, aber f�r GetCurrentDirectory hatte ich nunmal nur ein Beispiel mit TCHAR zur Hand

Code:

std::vector<wchar_t> buffer(MAX_PATH);
GetCurrentDirectoryW(buffer.size(), &buffer[0]);
std::wstring myDirectory(buffer.begin(), buffer.end());

Saubere C++-L�sung.

Quote:

Originally Posted by MrSm!th

p.s. Zum Teil auch deswegen, weil ich Arrays noch nie mochte xD

Solltest du auch nicht, wozu etwas statisches wenn es dynamische Containerklassen in C++ gibt ;)

Quote:

und bei CreateProcess konnte als zweiter Parameter CString nicht in LPWSTR konvertiert werden

Ich gehe mal von den STL-strings aus. LPWSTR ist nichts anders als ein typdef auf wchar_t*. Ein std::wstring gibt beim Aufruf der Memberfunktion c_str() einen solchen Zeiger zur�ck, allerdings einen konstanten der hier nicht gefragt ist. In diesem Fall solltest du eher einen Zeiger auf das erste Zeichen �bergeben:

Code:

std::wstring meinString("foobar");
CreateProcessW(, &meinString[0], ...);

CString ist etwas aus der MFC, oder? W�rde ich nicht unbedingt nutzen.

12/15/2009 22:13 MrSm!th#13

old post:

Ist aber sehr Praktisch und der Standardtyp von Editboxen ;)

Danke f�r die zwei L�sungen, werde sie dem n�chst mal probieren.
Und nochmals danke euch zwei, ohne euch w�r ich nicht weitergekommen xD
Habt auch Credits beim release bekommen ;)

new:

So. Entschuldigung, dass ich diesen etwas �lteren Thread wieder hoch holen muss, aber es gibt wieder mal ein kleines Problem mit CreateProcess und ich hoffe ich kann wieder so eine Hilfe wie beim letzten Mal bekommen.
Also:
Wir waren ja so weit, das Spiel zu starten, wenn der Launcher im selben Verzeichnis ist.
So nun bekam ich den Tipp, dass der Hackshield einen Registryeintrag mit seinem Verzeichnis macht.
Also wollte ich dessen Pfad auslesen und dann das HShield durch S4Client.exe ersetzen.
Ging auch super.
Nur gabs nen Error vom Client. Mittlerweile wei� ich warum:
Der Client wird wegen Parameter 8 =NULL im selben Verzeichnis wie der Launcher gestartet.
Das sollte kein Problem sein, wenn er im selben Verzeichnis ist, aber das soll ja gerade keine Bedingung sein.
Also habe ich eben aus dem Pfad den ich sowieso schon ausgelesen habe, auch noch den Pfad nur zum Verzeichnis ohne S4Client.exe ausgelesen.
War ja keine Schwierigkeit.
Da war das Konvertieren in LPCWSTR schon etwas schwerer, ging aber dank ein kleines bisschen C&P auch ohne Probleme.
Nur gibts eine Exception wegen Zugriffsverletzung.

(Und da bei google die meisten Beispiele mit dem Parameter=0 sind, habe ich kein �hnliches Problem+L�sung gefunden!)

Hier mal der n�tige Code:

PHP Code:


			
//Globale Variablen
std::string path;
int size;    // ich wei� nicht elegant, war aber an dem Abend gerade nicht am logischsten und habe auf einen R�ckgabewert verzichtet :D

void FindDir()
{
    //Pfad auslesen
    char buf[256];
    HKEY hKey;
    char temp[256];
    DWORD dwType = 0;
    DWORD dwBufSize = 255;
      if( RegOpenKeyEx(HKEY_CURRENT_USER,_T("Software\\Ahnlab\\HShield"),0,KEY_QUERY_VALUE,&hKey) == ERROR_SUCCESS)
    {
        dwType = REG_SZ;
        if(RegQueryValueEx(hKey,_T(""),NULL, &dwType,(LPBYTE)temp, &dwBufSize)==ERROR_SUCCESS)
        {

//hier beginnt der Teil zum kopieren ohne die Nullen, die einfach zwischen die Arrayelemente gesetzt werden, zu �bernehmen!
             int i =0;
             int i2 = 0;
             while(i <256)
             {
                buf[i2] = temp[i];
                i2++;
                i = i+2;
             }
//Ende
             path = buf;
             size = strlen(buf)-8; // wurd f�r das substr() gleich gebraucht und f�r replace()


        }
        else
        {
            CString str;
            str.Format(L"Error: %i",GetLastError());
            AfxMessageBox(str);
        }
        RegCloseKey(hKey);
    }
    else
      {
        CString str;
        str.Format(L"Error: %i",GetLastError());
        MessageBox(str,L"Error!",MB_OK);
    
      }
}

std::wstring s2ws(const std::string& s)
{
 int len;
 int slength = (int)s.length() + 1;
 len = MultiByteToWideChar(CP_ACP, 0, s.c_str(), slength, 0, 0);
 wchar_t* buf = new wchar_t[len];
 MultiByteToWideChar(CP_ACP, 0, s.c_str(), slength, buf, len);
 std::wstring r(buf);
 delete[] buf;
 return r;
}

void ClittleS4launcherDlg::OnBnClickedButton1()
{
ZeroMemory( &si, sizeof(si) );
    si.cb = sizeof(si);
    ZeroMemory( &pi, sizeof(pi) ); 


    //Dir + Namen anh�ngen
    FindDir();
    std::string str1 = "\\S4Client.exe\"";
    path.replace(size,str1.length(),str1);
    path = "\""+path;
    std::string tmp = path.substr(0,size+2);

    //Working Dir
    std::wstring stemp = s2ws(tmp); 
    LPCWSTR dir = stemp.c_str();

    //Konvertieren vom vollen Pfad
    const char* all = path.c_str();
    int len = 1 + strlen(all);
    wchar_t* t = new wchar_t[len];
    mbstowcs(t, all, len);
    TCHAR* path2 = (TCHAR*)t;

    TCHAR param1[] =L" -rc:eu -lac:eng -auth_server_ip:217.110.102.94";
    TCHAR param2[] =L" -rc:eu -lac:ger -auth_server_ip:217.110.102.94";


    if(lac == 1) //Englisch
    {
        _tcscat(path2,param1);
        if(!CreateProcess(NULL,path2,NULL,NULL,FALSE,0,NULL,dir,&si,&pi))
        {
            CString str;
            str.Format( L"Error: %i", ::GetLastError());
             MessageBox( str, L"Error!", MB_OK );
        }
    }
    else if(lac == 2) //Deutsch
    {        
        _tcscat(path2,param1);
        if(!CreateProcess(NULL,path2,NULL,NULL,FALSE,0,NULL,dir,&si,&pi))
        {
            CString str;
            str.Format( L"Error: %i", ::GetLastError());
             MessageBox( str, L"Error!", MB_OK );
        }
    }
    else
        AfxMessageBox(_T("Keine Sprache gew�hlt./ No Language selected!"));

}
}

12/15/2009 22:26 backo#14

Quote:

Originally Posted by MrSm!th

...

In deiner OnBnClickedButton1 funktion entsteht ein Memory leak. Da fehlt ein delete[] t; und denk dir mal bessere namen f�r die Variablen aus. Les mir mal den code in paar minuten durch.

12/15/2009 22:29 MrSm!th#15

hm.
und alleine deswegen?
der fehler kommt erst bei CreateProcess

(und solche mini-namen nehme ich gerne f�r Variablen die kruzzeitig genutzt werden/r�ckgabewerte sind (au�erdem ist gerade der teil C&Ped; ich habe nach einer guten Konvertierungsm�glichkeit gesucht und das gefunden^^) f�r wichtige nehme ich auch welche mit sinn)

edit: Ausnahme kommt immernoch.
Ich vermute, dass es an einem Overflow liegt, der durch Nutzen von _tcscpy anstatt _tcscpy_s kommt. Aber eigentlich habe ich daf�r gesorgt, dass alles passen m�sste und beim Debuggen werde alle String (auch nach _tcscpy) richtig angezeigt =/

Page 1 of 2