C++ Debug Assertion Failed

[Klemens102]

Moin Leute,

ich bin gerade dabei mir einen Hardware ID Generator
zu schreiben, ich mache das über die Serien Nummer der
Festplatte. Zum Test wollte ich mir jetzt einmal die
Seriennumer in einem Label ausgeben, wenn ich die erstellte
Form starte und auf Start klicke, erhalte ich folgende Fehlermeldung:



Hier die Source:

void GetSerialNumber(char *sDevice, char *Serial)
{
DWORD SerialNum;
GetVolumeInformationA(NULL, NULL, NULL, &SerialNum, NULL, NULL, NULL, NULL);
sprintf(buffer,"%d",SerialNum);
label1->Text = Convert::ToString(*buffer);
return;

}

Ich hoffe jemand kann mir helfen.

Gruß
//localhost

[SmackJew]

"Please Retry to debug the application", daran schon mal gedacht?

Zum Code: Woher kommt buffer? Und warum übergibst du "ToString" nur ein char? Kann mir kaum vorstellen, dass 'ne Methode die "ToString" heißt nur ein char bekommt.

[Klemens102]

Originally Posted by SmackJew

"Please Retry to debug the application", daran schon mal gedacht?

Zum Code: Woher kommt buffer? Und warum übergibst du "ToString" nur ein char? Kann mir kaum vorstellen, dass 'ne Methode die "ToString" heißt nur ein char bekommt.

Ich hab jetzt schon öfters Debuged.
Hab es auch schon im release versucht, da beendet sich das Programm dann halt ohne Fehlermeldung.

Der Buffer ist oben decladiert.

char HDD3[200];
char *disk = " :\\";
char Disk = 'c';
char *buffer;

[SmackJew]

Originally Posted by //localhost

Ich hab jetzt schon öfters Debuged.

Offenbar nicht erfolgreich. Was macht man wenn man scheitert? Nochmal probieren und dies mal richtig gucken und nachdenken.

Mit dem Crash hat das wahrscheinlich nichts zu tun, aber das hier:

label1->Text = Convert::ToString(*buffer);

wird dir genau 1 char liefern.

Zum Debuggen: Wenn du mit dem VS Debugger nicht zurecht kommst, nimm den den du lieber magst. Olly z.B.. Desweiteren ist der Fehler in sprintf.c, wahrscheinlich hat es also mit sprintf zu tun.

[Klemens102]

Originally Posted by SmackJew

Offenbar nicht erfolgreich. Was macht man wenn man scheitert? Nochmal probieren und dies mal richtig gucken und nachdenken.

Mit dem Crash hat das wahrscheinlich nichts zu tun, aber das hier:

label1->Text = Convert::ToString(*buffer);

wird dir genau 1 char liefern.

Zum Debuggen: Wenn du mit dem VS Debugger nicht zurecht kommst, nimm den den du lieber magst. Olly z.B.. Desweiteren ist der Fehler in sprintf.c, wahrscheinlich hat es also mit sprintf zu tun.

Hast du einen Vorschlag, wie ich das besser machen könnte ?

label1->Text = Convert::ToString(*buffer);

[SmackJew]

Originally Posted by //localhost

Hast du einen Vorschlag, wie ich das besser machen könnte ?

label1->Text = Convert::ToString(*buffer);

[Klemens102]

Originally Posted by SmackJew

Damit habe ich es ja schon versucht,
allerdings kriege ich es nicht auf die Reihe :/

Gibt es noch weitere Vorschläge ?

[MrSm!th]

Um dir mal nen Tipp zu geben, was er meint:
Lass das Sternchen weg, du übergibst schließlich so nur einen char und keinen String, weil du mit * den char* dereferenzierst.

Desweiteren: allozierst du überhaupt genügend Speicher? Ich sehe eine Deklaration von buffer, aber keine Speicherallokation!
Da ist nochmal Grundlagenwiederholung angesagt ;O

Oh und lass diesen .NET Form Dreck weg, der hat nichts in C++ zu suchen! Nimm lieber reind C++ Frameworks für GUIs, wobei die am Anfang ohnehin mehr schaden als nutzen.

Oh und eine HWID über GetVolumeInformation ist nicht besonders sicher.

[Klemens102]

Originally Posted by MrSm!th

Um dir mal nen Tipp zu geben, was er meint:
Lass das Sternchen weg, du übergibst schließlich so nur einen char und keinen String, weil du mit * den char* dereferenzierst.

Desweiteren: allozierst du überhaupt genügend Speicher? Ich sehe eine Deklaration von buffer, aber keine Speicherallokation!
Da ist nochmal Grundlagenwiederholung angesagt ;O

Oh und lass diesen .NET Form Dreck weg, der hat nichts in C++ zu suchen! Nimm lieber reind C++ Frameworks für GUIs, wobei die am Anfang ohnehin mehr schaden als nutzen.

Oh und eine HWID über GetVolumeInformation ist nicht besonders sicher.

Danke, deine Antwort hat mir geholfen.
Ich hatte vor einem Jahr Pause mit C++ gemacht und möchte das jetzt wieder als einstiegsprojeckt nutzen. Wie würdest du eine Hardware ID generieren?