Mysteriöses Speicherleck in C++ Datei Ausgabefunktion

Question

Ich hasse es, das alte "hier ist ein Code, was ist los damit?" aber ich habe dieses Problem mehrere Tage lang ohne Fortschritt untersucht. Dieser Code stürzt normalerweise sofort ab, wenn der Debugger ausgeschaltet ist, obwohl gelegentlich ein Speicherplatz von einigen Stunden zur Verfügung steht, in dem er korrekt kompiliert und ausgeführt wird. Mit angeschlossenem gdb funktioniert es, aber verliert Speicher bei ungefähr 10 MB pro Sekunde und stürzt schließlich ab, wenn es ausläuft.

Es ist definitiv diese Funktion; Solche Probleme treten nicht auf, wenn der einzige Aufruf an sie auskommentiert wird, und ein seltener Aufruf verzögert den Ausfall des Arbeitsspeichers.

//Write a frame of the animation to disk 
void draw_frame(int framenum) { 
    ofstream fout; 
    ostringstream fname; 
    fname << "D:\\frames\\" << framenum << ".data"; 
    fout.open(fname.str(), ios::binary | ios::out); 

    unsigned char ***frame; 
    frame = new unsigned char ** [WORLDSIZE]; 
    for (int x = 0; x < WORLDSIZE; x++) { 
     frame[x] = new unsigned char * [WORLDSIZE]; 
     for (int y=0; y < WORLDSIZE; y++) { 
      frame[x][y] = new unsigned char [3]; 
     } 
    } 

    unsigned long long ***colormix; 
    colormix = new unsigned long long ** [WORLDSIZE]; 
    for (int x = 0; x < WORLDSIZE; x++) { 
     colormix[x] = new unsigned long long * [WORLDSIZE]; 
     for (int y=0; y < WORLDSIZE; y++) { 
      colormix[x][y] = new unsigned long long [3]; 
      for (int z=0; z < 3; z++) { 
       colormix[x][y][z]=0; 
      } 
     } 
    } 

    for (vector<SmellyObject *>::iterator it = smellythings.begin(); it != smellythings.end(); ++it) { 
     SmellyObject *theobj = *it; 
     for (int x=0; x < WORLDSIZE; x++) { 
      for (int y=0; y < WORLDSIZE; y++) { 
       double scentlevel = scentmaps[theobj -> id][x][y]; 
       double colorlevel = (scentlevel/10000.0); 
       colormix[x][y][0] += theobj->r * colorlevel; 
       colormix[x][y][1] += theobj->g * colorlevel; 
       colormix[x][y][2] += theobj->b * colorlevel; 
      } 
     } 
    } 

    for (int x=0; x < WORLDSIZE; x++) { 
     for (int y=0; y < WORLDSIZE; y++) { 
      for (int z=0; z < 3; z++) { 
       //cout << colormix[x][y][z] << " "; 
       frame[x][y][z] = min(255.0, (colormix[x][y][z]/(double) smellythings.size())); 
      } 
     } 
    } 

    for (int x=0; x < WORLDSIZE; x++) { 
     for (int y=0; y < WORLDSIZE; y++) { 
      fout.write((char *) frame[x][y], 3); 
     } 
    } 

    fout.close(); 

    frame = new unsigned char ** [WORLDSIZE]; 
    for (int x = 0; x < WORLDSIZE; x++) { 
     frame[x] = new unsigned char * [WORLDSIZE]; 
     for (int y=0; y < WORLDSIZE; y++) { 
      for (int z=0; z < WORLDSIZE; z++) { 
       //delete[] &frame[x][y][z]; 
      } 
      delete[] frame[x][y]; 
     } 
     delete[] frame[x]; 
    } 
    delete[] frame; 

    colormix = new unsigned long long ** [WORLDSIZE]; 
    for (int x = 0; x < WORLDSIZE; x++) { 
     colormix[x] = new unsigned long long * [WORLDSIZE]; 
     for (int y=0; y < WORLDSIZE; y++) { 
      for (int z=0; z < WORLDSIZE; z++) { 
       //delete[] &colormix[x][y][z]; 
      } 
      delete[](colormix[x][y]); 
     } 
     delete[](colormix[x]); 
    } 
    delete[](colormix); 

    return; 
} 

WORLDSIZE 50 ist, und ist ein scentmapsstd::unordered_map Mapping ints (die .id Eigenschaft eines SmellyObject) x WORLDSIZE Arrays verdoppelt WORLDSIZE.

Answer 1

frame = new unsigned char ** [WORLDSIZE]; 

Sie ordnen dieses Array am Anfang der Funktion zu, und ordnen Sie alle seine drei Dimensionen zu, verschlingen eine große Menge an Speicher. später

Dann ...

fout.close(); 

frame = new unsigned char ** [WORLDSIZE]; 

... Sie neu zugewiesen nur frame zu einem anderen zugewiesenen Array. Der alte Zeiger frame ist verschwunden, und Sie haben gerade die metrische Tonne Speicher freigegeben, die Sie ursprünglich zugewiesen haben.

Derselbe Fehler mit Ihrem anderen, colormix Array.

Lektion, die daraus gelernt werden: Der beste Weg, um Bugs zu beheben, ist, sie nie an erster Stelle zu machen. Hätten Sie C++ - Container verwendet, die alle Speicherzuweisungen für Sie richtig handhaben, wie std::vector, wäre dies nie passiert. Richtiges Verwenden der vollen Ressourcen der C++ - Bibliothek - Container, Iteratoren, Algorithmen - machen es logisch unmöglich, dass viele häufige Programmierfehler passieren.