Variable aus malloc() auf 0 setzen

Wenn ich das ShowMessage ein wenig erweitere

int bytes = _rtl_write(handle,*var,filesize);
_rtl_close(handle);
char *errormsg = strerror(errno);

ShowMessage(FileName+"..."+IntToStr((__int64)filesize)+"..."+IntToStr(bytes)+".."+errormsg);

Dann sagt er mir bytes = -1 und bei errormsg "Invalid Argument" als Fehler. Aber wie gesagt, nur wenn ich ZeroMemory nutze. Ich schreibe die Daten immer bei Programmende. Wenn ich den Button nicht drücke, welcher ZeroMemory aufruft, dann habe ich keine Fehler und alle Daten werden erfolgreich geschrieben.

David_pb

Was für ein Typ hat "data" denn. Wie ist der Typ deklariert. Kannst du ein funktionierendes Minimalbeispiel liefern das den Fehler repräsentiert. Ich seh kein ZeroMemory im Code... Das wars erstmal!

Also ZeroMemory steht in meinem Code ja oben, das würde als Kompletter Code nur so ausschauen

void __fastcall TForm1::Button3Click(TObject *Sender)
{
    ZeroMemory(&data,sizeof(data));
}

Und die Deklaration und Zuweisung steht hier oben etwas ausführlicher:
http://www.c-plusplus.net/forum/viewtopic-var-p-is-1955192.html#1955192

Sorry für die Doppelposts ^^°

Nur um das vielleicht nochmal zu verdeutlichen. Das hier funktioniert einwandfrei. Ich würde das nur gern schneller haben und dachte dafür sei ZeroMemory gedacht.

void __fastcall TForm1::Button3Click(TObject *Sender)
{
    char empty = 0;
    for (int i=0;i<55321;i++) // 55321 ist die Dateigröße in Bytes
    {
    	memcpy(&data[i],&empty,1);
    }
}

_matze

~zero schrieb:

ZeroMemory(&data,sizeof(data));

Wieso der Adressoperator? Du übergibst hier einen Zeiger auf einen Zeiger. Damit setzt du deine Zeiger-Variable selbst auf 0 (und verlierst den Zugriff auf den reservierten Speicher). Also lass den mal weg. Was als 2. Parameter übergeben werden soll, kann dir keiner sagen, weil du den malloc-Aufruf nicht gepostet hast. sizeof(data) ist jedenfalls falsch (das ist die Länge vom Zeiger, nicht vom Speicher, auf den der Zeiger zeigt).

_matze schrieb:

Was als 2. Parameter übergeben werden soll, kann dir keiner sagen, weil du den malloc-Aufruf nicht gepostet hast.

Das steht oben. Vielleicht habe ich zuviel Code rein gestellt und wurde übersehen. Aber wie gesagt, data beinhaltet eine Datei und damit eine Dateigröße.

filesize=filelength(handle)
*var = (unsigned char*) realloc(*var,filesize+1);

Siehe ReadFile()

_matze

~zero schrieb:

_matze schrieb:

Was als 2. Parameter übergeben werden soll, kann dir keiner sagen, weil du den malloc-Aufruf nicht gepostet hast.

Das steht oben. Vielleicht habe ich zuviel Code rein gestellt und wurde übersehen. Aber wie gesagt, data beinhaltet eine Datei und damit eine Dateigröße.
filesize=filelength(handle)
*var = (unsigned char*) realloc(*var,filesize+1);
Siehe ReadFile()

Ok. Klappt es denn nun ohne Adressoperator?

_matze schrieb:

Wieso der Adressoperator? Du übergibst hier einen Zeiger auf einen Zeiger. Damit setzt du deine Zeiger-Variable selbst auf 0 (und verlierst den Zugriff auf den reservierten Speicher). Also lass den mal weg.

Ups, Copy&Waste ^^ Ja stimmt natürlich, jetzt schreibt er die Variable endlich auch in die Datei. Danke!

Wenn ich die Dateilänge als size mit angebe, dann funktioniert es endlich wie gewollt. Die Möglichkeit aus data die Bytegröße zu bestimmen existiert ja sicherlich nicht, oder?

_matze

~zero schrieb:

Die Möglichkeit aus data die Bytegröße zu bestimmen existiert ja sicherlich nicht, oder?

Nein, in dem Fall ist data nur ein Zeiger, der dir leider nicht verrät, wie groß der Speicherbereich ist, auf den er zeigt.

Danke für die Hilfe!

Wutz

~zero schrieb:

Also ZeroMemory steht in meinem Code ja oben, das würde als Kompletter Code nur so ausschauen
void __fastcall TForm1::Button3Click(TObject *Sender)
{
    ZeroMemory(&data,sizeof(data));
}

Das hieße, dass data eine globale Variable ist, was schlechter Stil ist, nicht thread-safe ist, ...

Wutz

~zero schrieb:

Die Möglichkeit aus data die Bytegröße zu bestimmen existiert ja sicherlich nicht, oder?

Wenn du MSVC/MinGW verwendest, geht

_msize(data)

für zuvor mit *alloc angeforderten Speicherbereich, ist aber kein ANSI C, wie das meiste, was du hier schreibst.