folgender Codeschnippsel:

// Get all icon objects from the vector and write its names into the XML file
	for (int i=1; i< numberOfIcons; i++)
	{
		string temp = myIcons.getIconVector().at(i)->getName();
		char *buf = new char[strlen(temp.c_str())];

		strcpy(buf, temp.c_str());

		XML.write(buf);

		//...When you are done with buf...
		delete []buf;

	}

Verursacht eine Heap-Corruption an der delete []buf;

Hat jemand eine Idee, warum?

gruß
Alex

Ansonsten, deletest du zwei Mal, oder führst du sonstige Speicherzugriffe durch?

Anscheinend lag es doch daran, mit :

char *buf = new char[strlen(temp.c_str())+1];

funktioniert es. Aber warum verstehe ich nicht

Aber warum verstehe ich nicht

Steht aber in meinem oberen Post. Du solltest schon genug Speicher anfordern, sonst wird das '\0' irgendwo hingeschrieben.

wenn diese string-variable ein std::string ist, dann wäre folgender code besser:

// Get all icon objects from the vector and write its names into the XML file
	for (int i=1; i< numberOfIcons; i++)
	{
            string temp = myIcons.getIconVector().at(i)->getName();
            XML.write(temp.c_str());
	}

Achso und delete versucht dann quasi einen zu langen Speicherabschnitt zu löschen?

Nein, das delete[] löscht so viele Bytes, wie mit new[] angefordert wurden (vorausgesetzt, du hast einen gültigen Zeiger).

strcpy() ist das Problem, da es alle char -Elemente inklusive '\0' kopiert und nicht darauf achtet, ob genügend Speicher vorhanden ist oder nicht.

Das ist übrigens eines der Probleme von C, in C++ kann man das mit dem Datentyp std::string umgehen.

{
        string temp = myIcons.getIconVector().at(i)->getName();
        vector<char> buf(temp.c_str(), temp.c_str() + temp.size() + 1);

        XML.write(&buf[0]);

        //...When you are done with buf...
        // ... NOTHING, 'cause this is C++
    }