finde das Speicherleck nicht

pospiech

Ich habe Simulationsdaten die in einem Array fester Größe gespeichert werden.
Das was darin fertigberechnet wurde wird an eine Plot Klasse übergeben und dort als Kopie gespeichert. Davon wird dann intern vom mit einer 'clone'-Funktion nochmal eine Kopie erstellt.

Ich habe jetzt den Verdacht das die Kopie von clone nicht gelöscht wird - weiß aber nicht wie ich das bewerkstelligen sollte. Aber vielleicht ist das Leck ja noch woanders?

Ich komme darauf, weil das new in setData zweimal pro übergeben von Daten ausgeführt wird, das delete aber nur einmal. Im folgenden der relevante Code:

Holen der Daten, übergeben an Plot und löschen der geholten Daten:

void MainWindow::updateHistory()
{
	int SizeX;
	int SizeY;

	double * TimeArray  = Laser->simulation.getTimeArray(SizeX, SizeY);

	SpectrogramPlot1->setData(TimeArray, SizeX, SizeY); 
	SpectrogramPlot1->qwtPlot->replot();

	delete [] TimeArray;
}

Holen der Daten aus internem Speicher

double * SimulationData::getTimeArray(int & SizeX, int & SizeY)
{
	SizeX = SaveIteration;
	SizeY = ArrayDataPoints - 1;
	double * Array = new double[SizeX * SizeY];
	memcpy(Array, TimeAmplitudeArray, SizeX * SizeY * sizeof(double));
	return Array;
}

weiterreichen im Plot

void QSpectrogramPlot::setData(double * Array, int sizex, int sizey)
{
	m_RasterData.setData(Array, sizex, sizey); // SpectrogramData
...
}

SpectrogramData Klasse inlkusive setData und clone

class SpectrogramData: public QwtRasterData
{
private:
	double * m_Array;
...

public:
	// Constructor giving back the QwtRasterData Constructor
    SpectrogramData(): QwtRasterData()
    {
		m_Array = NULL;
    }

    ~SpectrogramData()
    {
		if (m_Array != NULL)
			delete [] m_Array;
    }

    virtual QwtRasterData *copy() const
    {
        SpectrogramData *clone = new SpectrogramData();
		clone->setRangeX(m_RangeX.min, m_RangeX.max);
		clone->setRangeY(m_RangeY.min, m_RangeY.max);
		clone->setBoundingRect(QwtDoubleRect(m_RangeX.min, m_RangeY.min, m_RangeX.max, m_RangeY.max));
		clone->setData(m_Array, m_DataSize.x, m_DataSize.y);
		return clone;
    }

    void setData(double * Array, int sizex, int sizey)
    {
        m_DataSize.x = sizex;
        m_DataSize.y = sizey;
        int size = sizex * sizey;
        MinMaxArrayValue(Array, size, &m_minValue, &m_maxValue);
        if (m_Array != NULL)
            delete [] m_Array;
        m_Array = new double [size];
        memcpy(m_Array, Array, size * sizeof(double));
    }

vlad_tepesch

falls du visual studio verwendest, sieh dir mal den visual leak detector an.
Der lioefert dir im Debugmode, nach dem Beenden einen callstack zu allen leaks.

pospiech

vlad_tepesch schrieb:

falls du visual studio verwendest, sieh dir mal den visual leak detector an.
Der lioefert dir im Debugmode, nach dem Beenden einen callstack zu allen leaks.

Jam ich nutze VS, aber ich sehe nicht wo ich diese Daten einsehen könnte. (Ich benutzt eine deutsche Version.)

ich sehe das leck nicht auf anhieb, vielleicht bist du aber an allgemeinen verbesserungsvorschlägen interessiert?
das problem wäre nicht entstanden, wenn das ganze in C++ statt in "C mit klassen" programmiert wäre. <vector> macht die ganze drecksarbeit für dich und du musst dich nicht mit new und delete rumärgern.

pospiech schrieb:

if (m_Array != NULL)
			delete [] m_Array;

die prüfung kannst du dir sparen, delete auf NULL ist per definition erlaubt.

such mal im msdn nach "leak report" und/oder "crt debugging". du kannst die laufzeitbibliothek anweisen, zum programmende alle leaks aufzulisten.

pospiech

ich habe folgendes gefunden, aber das soll nur für pures C gelten und erzeugt bei mir auch gar keine Ausgabe:
http://msdn.microsoft.com/en-us/library/e5ewb1h3.aspx

The primary tools for detecting memory leaks are the debugger and the C Run-Time Libraries (CRT) debug heap functions. To enable the debug heap functions, include the following statements in your program:
Copy Code
#define _CRTDBG_MAP_ALLOC
#include <stdlib.h>
#include <crtdbg.h>

_CrtSetDbgFlag(_CRTDBG_LEAK_CHECK_DF);

ganz am anfang deiner main() bzw. WinMain() einfügen. funktioniert natürlich nur im debug-build!

pospiech schrieb:

aber das soll nur für pures C gelten

nein, das hast du falsch verstanden.

pospiech

naseweis schrieb:

_CrtSetDbgFlag(_CRTDBG_LEAK_CHECK_DF);
ganz am anfang deiner main() bzw. WinMain() einfügen. funktioniert natürlich nur im debug-build!

folgendes Kompiliert bei mir allerdings nicht:

#define _CRTDBG_MAP_ALLOC
#include <stdlib.h>
#include <crtdbg.h>

_CrtSetDbgFlag(_CRTDBG_LEAK_CHECK_DF);

int main(int argc, char** argv)
{

1>.\src\main.cpp(8) : error C4430: Fehlender Typspezifizierer - int wird angenommen. Hinweis: "default-int" wird von C++ nicht unterstützt.
1>.\src\main.cpp(8) : error C2365: "_CrtSetDbgFlag": Erneute Definition; vorherige Definition war "Funktion".
1> C:\Programme\Microsoft Visual Studio 8\VC\include\crtdbg.h(981): Siehe Deklaration von '_CrtSetDbgFlag'

naseweis schrieb:

pospiech schrieb:

aber das soll nur für pures C gelten

nein, das hast du falsch verstanden.

kann ich nicht beurteilen, steht so im Kommentar unten auf der Seite.

lies ein anfängertutorial.

drakon

pospiech schrieb:

...

In die main.. Du machst da einen Funktionsaufruf vor der main..

pospiech

naseweis schrieb:

lies ein anfängertutorial.

zu was? Compiler Macros von Visual Studio/Microsoft? Nicht sehr hilfreich so ein Hinweis.

EDIT:

drakon schrieb:

pospiech schrieb:

...

In die main.. Du machst da einen Funktionsaufruf vor der main..

Ja klar. Danke für den Hinweis. Ich arbeite normalerweise nie innerhalb von main.cpp...

Es wird übrigens mit dem code im debug Modus nie eine Ausgabe erstellt.

startest du das programm im debugger? schaust du dir den inhalt des ausgabefensters an?

drakon

pospiech schrieb:

Ja klar. Danke für den Hinweis. Ich arbeite normalerweise nie innerhalb von main.cpp...

Das hat mit dem nichts zu tun. Du hast einen Funktionsaufruf ausserhalb einer Funktion gemacht und das geht nun mal einfach nicht.

pospiech

naseweis schrieb:

startest du das programm im debugger? schaust du dir den inhalt des ausgabefensters an?

Ja. Ich schaue mir im Teilfenster 'Ausgabe' den Tab 'Ausgabe' an, der nach dem Starten des Programms folgendes enthält:

"ModelockingSimulationd.exe": "C:\Programme\Trillian\events.dll" geladen, Keine Symbole geladen.
"ModelockingSimulationd.exe": "C:\Programme\Trillian\msvcr71.dll" geladen, Keine Symbole geladen.
Der Thread 'Win32 Thread' (0xd7c) hat mit Code 0 (0x0) geendet.

die leaks werden erst beim beenden des programms ausgegeben. ansonsten fällt mir nix weiter ein, es müsste so eigtl. funktionieren.

pospiech

naseweis schrieb:

die leaks werden erst beim beenden des programms ausgegeben. ansonsten fällt mir nix weiter ein, es müsste so eigtl. funktionieren.

Dann steht da auch nicht viel mehr.

R3dNeXX

dann muss ich da mal blöd fragen: Woher weißt du denn, dass deine Applikation Leaks hat ??

Wenn im Output nix steht, dann dürfte es eigentlich keine Leaks geben (glaub ich zumindest )

Nexus

Bau doch mal ein Leak ein, zum Beispiel

new double;

Dann müsste dort nachher stehen "Leaks detected! 8 Byte in Zeile .." oder so ähnlich.

pospiech

R3dNeXX schrieb:

dann muss ich da mal blöd fragen: Woher weißt du denn, dass deine Applikation Leaks hat ??

weil sie mit jedem Plot (jede Sekunde) ca. 2 MB Speicher verbraucht und dieser Verbrauch steil nach oben geht. In wenigen Minuten kommen da 100te von MB zusammen.

Folgender Code liefert mir noch immer keine Ausgabe:

//#include <qt/qapplication.h>
//#include "MainWindow.h"

#define _CRTDBG_MAP_ALLOC
#include <stdlib.h>
#include <crtdbg.h>

int main(int argc, char** argv)
{
	_CrtSetDbgFlag(_CRTDBG_LEAK_CHECK_DF);

	//QApplication app( argc, argv );

	//// create a new instance of MainWindow
	//MainWindow mainWindow;	
	//mainWindow.show();

	//// Enters the main event loop and waits until exit() is called 
	//// or the main widget is destroyed, and Returns the value that 
	//// was set via to exit() (which is 0 if exit() is called via quit()). 
	//return app.exec();

	double * abc = new double;
}

Nexus

Schreib nach #include <crtdbg.h> Folgendes:

#define new new(_NORMAL_BLOCK, __FILE__, __LINE__)

Zudem kannst du _CrtSetDbgFlag() noch das Flag _CRTDBG_ALLOC_MEM_DF übergeben.

pospiech

Ok, das geht jetzt:

define _CRTDBG_MAP_ALLOC
#include <stdlib.h>
#include <crtdbg.h>
#define new new(_NORMAL_BLOCK, __FILE__, __LINE__)

int main(int argc, char** argv)
{
	_CrtSetDbgFlag(_CRTDBG_LEAK_CHECK_DF | _CRTDBG_ALLOC_MEM_DF);

	double * abc = new double; 
}

liefert mir

Detected memory leaks!
Dumping objects ->
.\src\main.cpp(24) : {138} normal block at 0x003F6B98, 8 bytes long.
Data: < > CD CD CD CD CD CD CD CD
Object dump complete.

Wenn ich das im eigentlichen Programm laufen lasse. Also im einfachsten Fall dieses einmal starte und direkt wieder beende, dann bekomme ich eine sehr lange Liste von solchen Lecks.

Wie kann ich mir anzeigen lassen zu welchen Dateien / Variablen die gehören?

Nexus

pospiech schrieb:

Wie kann ich mir anzeigen lassen zu welchen Dateien / Variablen die gehören?

Du siehst ja, in welcher Zeile und welcher Datei das Leak aufgetreten ist, dann kannst du den Fehler dort suchen.