0xC0000005: Access Violation nach CreateProcess - wieso?

Martin Richter

Dieser Code ist die Ursache:

char strCmd[] = ""; 
sprintf(strCmd, _T("%s"), strInhalt);

Du kopierst in enen String der 1 Zeichen lang ist mehr hinein!

vista

estartu schrieb:

char strCmd[] = "";
		sprintf(strCmd, _T("%s"), strInhalt);

typischer anfängerfehler, strCmd[] ist nur ein byte gross

estartu

Mist, wieso hat es überhaupt funktioniert?
Ich hatte das noch im Verdacht, sah im Debugger aber gut aus.

Okay, wie geht es richtig? GetBuffer will nämlich auch nicht.

Ist übrigens nicht wirklich ein Anfängerfehler, nur eine weitere Episode eines jahrelange Kampfes mit C-Zeichenketten.

CStoll

Eigentlich sollte GetBuffer() der richtige Ansatz sein, wie hast du es denn verwendet?

vista

etwa so...

BOOL fReturn = CreateProcess(NULL, strInhalt.GetBuffer(_MAX_PATH), ...);

estartu

Versucht habe ich:

char strCmd[] = strInhalt.GetBuffer(strInhalt.GetLength());

das gab aber

error C2440: 'initializing' : 'char *' kann nicht in 'char []' konvertiert werden
Es gibt keine Konvertierungen von Arraytypen, obwohl es Konvertierungen von Verweisen oder Zeigern in Arrays gibt

Aber das von vista funktioniert.

Dankeschön!

PS: Wenn ich meine Lösung direkt in den Funktionsaufruf schreibe, funktioniert die auch.
Wieso das blos wieder?

dEUs

ReleaseBuffer nicht vergessen!

@estartu: bei solchen Fehlern kann es interessant sein mit F11 zu steppen statt mit F10. So steppst du nämlich auch in die aufgerufenen Destruktor hinein und kannst so möglicherweise herausfinden, in welcher Klasse es knallt.

CStoll

Die Größe eines Arrays muß zur Compilezeit feststehen (wenn du ein String-Literal übergibst, kann der Compiler daraus die nötige Größe berechnen), mit einem Zeiger ( char*strCmd=...; ) wäre dir das nicht passiert.

dEUs

estartu schrieb:

PS: Wenn ich meine Lösung direkt in den Funktionsaufruf schreibe, funktioniert die auch.
Wieso das blos wieder?

Naja, weil diese Funktion eben einen char* erwartet. Was du gemacht hast ist eh Quatsch, wieso hast du strCmd überhaupt als char[] deklariert und nciht als char*?

estartu

Ich hab das so, weil es an anderer Stelle so mal funktioniert hat.
Wie man merkt, stehe ich mit dem Code auf Kriegsfuß und nehme da meine schon fertige Lösung, sofern vorhanden. (Das kann durchaus auch hier aus dem Forum sein...)

vista

estartu schrieb:

PS: Wenn ich meine Lösung direkt in den Funktionsaufruf schreibe, funktioniert die auch.
Wieso das blos wieder?

weil es einen unterschied zwischen pointer und array gibt.
du kannst den inhalt eines arrays nicht ändern oder es initialisieren, indem du ihm einen pointer zuweist.
wenn du bei einer fuktion, die einen char* erwartet, einen array-bezeichner einsetzt, dann wird automatisch die adresse des ersten element des arrays übergeben (und nicht das array selbst).

huch? ich bin ja viel zu spät

vista

dEUs schrieb:

ReleaseBuffer nicht vergessen!

das braucht man nur, wenn man den inhalt des CStrings über den pointer verändern will, den GetBuffer() lieferte.
in estartus fall ist es nicht nötig.

Martin Richter

Das ist eigentlich nicht das Problem. Hier wird ja der Stack überschrieben. Es müsste also sofort auffallen, das die Varoablenund der Callstack futssch sind.

Es müsste noch mehr auffallen, da in der Debugversion sofort die Frame-Guard Checker anspringen und einen ASSERT feuern!