Почему куча коррупции при возврате объекта Mat?

Question

Я не понимаю, почему я получаю поврежденную ошибку кучи с помощью этой программы (я использую OpenCV для класса Mat):

class A { 
    private: 
    Mat image;  

    static UINT ThreadProc(LPVOID pParam) { 
     A* pThis= (ClientNetwork*)pParam; 
     UINT nRet= pThis->DoThreadProc();  // get out of 'static mode' 
     return(nRet); 
    } 
    UINT ClientNetwork::DoThreadProc() { 
     vector<uchar> vect; 
     while(1) { 
      /**** initialize vect and get the image data to decode ****/ 

      decode(vect); 
     } 
    } 

    public: 
    void decode(const vector<uchar>& vectorData){image=imdecode(vectorData, CV_LOAD_IMAGE_COLOR);} 
    Mat get_image(){return image;} 
    void start() {m_pcThread= AfxBeginThread(ThreadProc, this);} 
} 

int main() { 
    A* a = new A(); 
    a->start(); 
    while(1) { 
     Mat image = a->get_image(); 
    } 
    delete a; 
    return 0; 
} 

кажется, что ошибка приходит из Mat image = a->get_image();, потому что если я возвращать ссылку вместо копии объекта, у меня нет ошибки больше:

Mat* get_image(){return ℑ} 

и

Mat* image = a->get_image(); 

I г ead, что возвращение копии объекта более элегантно на C++, чем ссылка. Поэтому я хотел бы знать, что не так.

EDIT: Visual studio разбивается на a->decode(vect), но это происходит только тогда, когда я возвращаю объект, а не ссылку.

EDIT 2: Я отредактировал код, чтобы отразить всю программу. Я думаю, проблема возникает из общего объекта a, который копируется и модифицируется одновременно. Я увижу, если проблема все еще происходит с использованием мьютекса.

Answer 1

Конструктор копирования cv::Mat не создает глубокую копию изображения. Он просто создает ссылку на оригинал Mat и увеличивает его счетчик ссылок. В следующей функции класса, return оператор вызывает конструктор копирования, возвращая ссылку на исходный image, который является вероятной причиной кучи коррупции:

Mat get_image(){ return image; } 

Вы должны вернуть глубокую копию image следующим образом, так что исходный код image не изменяется случайно.

Mat get_image(){ return image.clone(); } 

Answer 2

Вы используете a без его инициализации.

int main() { 
    A* a; 
    vector<uchar> vect; 
    while(1) { 
     // get the vector of data 
     a->decode(vect); 

Добро пожаловать в неопределенное поведение, население: вы.

Инициализация a для лучшего результата.

Answer 3

Тогда это проблема синхронизации потоков, как вы сами предложили. Изображение постоянно заполняется в этом бесконечном цикле while. И в середине его где-то вы пытаетесь сделать его копию. Путь к катастрофе. Вам нужно сделать блокировку записи внутри этого цикла в DoThreadProc на каждой итерации. И тогда вам нужно взять блокировку чтения внутри вашего get_image. Вам нужно использовать библиотеку блокировки чтения/записи, которая не изгоняет читателей.

Альтернативно вы можете работать с мьютексами/критическими разделами. Оба цикла записи и чтение (get_image) получают эксклюзивный доступ к изображению, пока они выполняют свою работу.

Мне любопытно, хотя - ваша процедура потока декодирует вещи в бесконечном цикле. Что вы пытаетесь сделать там? И при чтении какого изображения вы ожидаете? Любое изображение в этот момент времени в итерации цикла?