ich sitze schon seit Montag an ein und demselben C++-Problem, welches ich kurz schildern möchte.

Situation:
HINWEIS: alles ist stark gekürzt

• DLL A: erstellt mit Borland C++ Builder 6; wird von Host-Anwendung geladen; unbedingt notwendig wegen hierin realisierter GUI
• DLL B: erstellt mit VS 2005 Pro; wird von DLL A nachgeladen; einige Funktionen exportiert
• Daten werden über Zeiger auf struct mit Basis-Datentypen zwischen beiden ausgetauscht
• struct sieht so aus:

[cpp]struct stParameter
      {
          char cParameterName[MAXPARLENGTH];
          float fParameterValue;
          ULONG ulCountCallPath; // Anzahl der Elemente des Parameter-Aufrufpfades
          ULONG *pulCallPath; // Zeiger auf Parameter-Aufrufpfad
          stParameter()
          {
                  memset(&cParameterName[0],   0, sizeof(cParameterName));
                  fParameterValue = 0.0F;
                  ulCountCallPath = 0;
                  [b]pulCallPath = NULL;[/b]
              }
          ~stParameter()
          {
              if (this->pulCallPath)
              {
                              [b]delete []this->pulCallPath;[/b]
                              this->pulCallPath = NULL;
              };
          };
      };[/cpp]

• in DLL A wird eine Funktion von DLL B aufgerufen:

[cpp]for(i = 0; [...] )
      {
        stParameter *pParList = [b]new[/b] stParameter[iParametersCount];

        int iGDFP = pDA->dGParFP(pList[i].ulD, pParList, iParametersCount);
        if (iGDFP != -1)
        {
            for (int j = 0; j < iParametersCount; j++)
            {
                 stParameter *Para = [b]new[/b] stParameter(&pParList[j]);

                 ListItem = lvAvailablePar->Items->Add();
                 ListItem->Caption = Para->cParameterName;
                 // [...]
            }
            else
            {
                 sError = "Fehlermeldungstext";
            }

            if (pParList)
            {
                 [b]delete []pParList;[/b]
                 pParList = NULL;
            }
        }
       }[/cpp]

• der fragliche Teil der aufgerufenen Funktion in DLL B

[cpp]Para->pulCallPath = [b]new[/b] ULONG[Para->ulCountCallPath];[/cpp]

Problem:
Alles klappt soweit gut, aber beim Löschen von pParList kommt es dann zum Fehler, wenn im Destruktor der Zeiger gelöscht werden soll. Wenn ich das Löschen auskommentiere, klappt wieder alles.

Fragen:

1. Ist der Destruktor von stParameter so korrekt bzw. wird durch Funktion in DLL B der Zeiger im struct dynamisch?
2. Wenn ich nicht den Speicher freigebe, entsteht dann ein memory leak?

~stParameter()
{
    delete[] this->pulCallPath;
}

Gleiches gilt auch für die andere Stelle, an der Du Speicher per 'delete[]' freigibst. Insbesondere der Test davor ist überflüssig.

Und dann sollte sich immer der Bereitsteller des Speichers um die Freigabe kümmern, oder aber explizit die Besitzherrschaft übertragen. Woher z.B. weiß Deine Klasse, dass ein Array freigegeben werden muss? Der Rest ist ziemlich chaotisch.

Insbesondere der Test davor ist überflüssig.

Die Prüfung vor Freigabe des Speichers werde ich entfernen.

Und dann sollte sich immer der Bereitsteller des Speichers um die Freigabe kümmern, oder aber explizit die Besitzherrschaft übertragen. Woher z.B. weiß Deine Klasse, dass ein Array freigegeben werden muss? Der Rest ist ziemlich chaotisch.

Das wirkliche Problem besteht darin, dass in DLL B ein Vektor im Vektor in ein Array gewandelt wird. Die Größe vom Array kann erst zur Laufzeit bestimmt werden. Sonst würde ich das ganze statisch machen und hätte das Problem mit delete nicht.

Der Zeiger im struct ist ja an sich statisch. Wenn ich in DLL B die unten aufgeführte Codezeile ausführe, muss dann der Speicher des Zeigers pulCallPath im struct dann freigegeben werden?

Wenn ich nicht den Speicher freigebe, entsteht dann ein memory leak?

Überprüf das doch auch mal mit einem Analysis Tool. z.B
http://valgrind.org/

Ansonsten gilt die Grundregel, dass du jeden Speicher, den du mit new/new[] anforderst auch wieder mit delete/delete[] freigeben musst.

Danke für die Vorschläge und Hilfe.