Błąd pamięci podczas korzystania z memcpy?

Question

Używam biblioteki dcmtk do modyfikowania danych pikseli skompresowanego obrazu dicom o wielu klatkach. Tak więc, aby to zrobić, w pewnym momencie w pętli for biorę dane pikselowe każdej zdekompresowanej ramki i modyfikuję je zgodnie z moim życzeniem i próbuję połączyć poszczególne zmodyfikowane dane pikseli w dużym buforze pamięci klatka po klatce. Ten podstawowy proces pętli for jest następujący.

Problem jest po pierwszej iteracji, która daje pamięć w wierszu kodu, w którym nazywam funkcję getUncompressedFrame. Myślę, że dzieje się tak z powodu linii memcpy(fullBuffer+(i*sizeF),newBuffer,sizeF);, ponieważ kiedy usuwam tę linię, nie ma w tym czasie błędu, a cała pętla for działa absolutnie dobrze.

Czy możesz mi powiedzieć, jeśli popełniam błąd podczas pracy z memcpy? Dzięki.

Uint32 sizeF=828072;// I just wrote it to show what is the data type. 
Uint8 * fullBuffer = new Uint8(int(sizeF*numOfFrames));//The big memory buffer 
for(int i=0;i<numOfFrames;i++) 
{ 
    Uint8 * buffer = new Uint8[int(sizeF)];//Buffer for each frame 
    Uint8 * newBuffer = new Uint8[int(sizeF)];//Buffer in which the modified frame data is stored 
    DcmFileCache * cache=NULL; 
    OFCondition cond=element->getUncompressedFrame(dataset,i,startFragment,buffer,sizeF,decompressedColorModel,cache); 
    //I get the uncompressed individual frame pixel data 
    if(buffer != NULL) 
    { 
     for(unsigned long y = 0; y < rows; y++) 
     { 
      for(unsigned long x = 0; x < cols; x++) 
      { 
       if(planarConfiguration==0) 
       { 
        if(x>xmin && x<xmax && y>ymin && y<ymax) 
        { 
         index=(x + y + y*(cols-1))*samplePerPixel; 
         if(index<sizeF-2) 
         { 
          newBuffer[index] = 0; 
          newBuffer[index + 1] = 0; 
          newBuffer[index +2] = 0; 
         } 
        } 
        else 
        { 
         index=(x + y + y*(cols-1))*samplePerPixel; 
         if(index<sizeF-2) 
         { 
          newBuffer[index] = buffer[index]; 
          newBuffer[index + 1] = buffer[index + 1]; 
          newBuffer[index + 2] = buffer[index + 2]; 
         } 
        } 
       } 
      } 
     } 
     memcpy(fullBuffer+(i*sizeF),newBuffer,sizeF); 
     //concatenate the modified frame by frame pixel data 
    }     

Answer 1

Zmień deklaracja fullBuffer do tego:

Uint8 * fullBuffer = new Uint8[int(sizeF*numOfFrames)]; 

Twój kod nie przydzielić tablicę, to przydzielony jeden Uint8 z wartością int(sizeF*numOfFrames).

Answer 2

Jeśli metoda getUncompressedFrame robi wewnętrzny memcpy do pamięci podręcznej, to ma sens, ponieważ podajesz wskaźnik zerowy jako argument dla pamięci podręcznej, bez przydzielonej pamięci.

Answer 3

Uint8 * fullBuffer = new Uint8(int(sizeF*numOfFrames)); 

to przydziela jeden bajt nadanie jej początkowej wartości sizeF*numOfFrames (po obcięcie najpierw do int a następnie Uint8). Chcesz tablicę, a ty nie chcesz obciąć wielkości do int:

Uint8 * fullBuffer = new Uint8[sizeF*numOfFrames]; 
          ^    ^

lub naprawić prawdopodobnych przecieki pamięci w kodzie:

std::vector<Uint8> fullBuffer(sizeF*numOfFrames);