Ich greife imit einem thread auf ein CImage objekt zu und zeichne dieses gleichzeitig in einem dialogfenster und mein prog stürzt regelmäsig ab:

Ich habe in der Doc-klasse eine menbervariable

CImage m_bmp;

die ich im Konstruktor erzeuge

m_bmp.Create(800,800,24);

Dann hab ich einen Thread, in dessen funktion auf dieses image zugegriffen wird,
z.B. so:

UINT __cdecl Cradar_02App::Thread708Func(LPVOID pParam){
	ThreadStruct* temp=(ThreadStruct* )pParam;
	Cradar_02Doc* pDoc=(Cradar_02Doc*)temp->pDoc;
	while(true){	
		pDoc->m_pbmp.SetPixel(100,100,RGB(222,222,222));
	}	
	return 0;	
}

Ausserdem zeichne ich das image in einem dialog:

void CPaintDlg::OnPaint(){
	Cradar_02Doc* pDoc =        dynamic_cast<Cradar_02Doc*>(((CMainFrame*)AfxGetMainWnd())->GetActiveDocument());
	ASSERT(pDoc);
	// Gerätekontext zum Zeichnen
    CPaintDC dc(this); 
    CPaintDC* pdc=&dc;

    CRect rRect;
    GetClientRect(rRect);
    rRect.NormalizeRect();

    pDoc->m_pbmp.Draw(pdc->m_hDC, rRect);

    DeleteObject(&pdc);
    DeleteObject(&rRect);
}

Wenn jetzt der thread läuft und man das fenster schnell hin und her bewegt (sodass oft onpaint aufgerufen wird) stürzt das programm durch einen ASSERT in ReleaseDC() ab:

Der Debugger zeigt mir dann an:

inline void CImage::ReleaseDC() const throw()
{
	HBITMAP hBitmap;

	ATLASSUME( m_hDC != NULL );

	m_nDCRefCount--;
	if( m_nDCRefCount == 0 )
	{
		hBitmap = HBITMAP( ::SelectObject( m_hDC, m_hOldBitmap ) );
->		ATLASSERT( hBitmap == m_hBitmap );
		s_cache.ReleaseDC( m_hDC );
		m_hDC = NULL;
	}
}

hBitmap ist auf 0x00000000 ?!?

Vielleicht weiss ja jemand, was ich hier verbocke.

Es wundert nicht, dass dies aus zwei Threads heraus und einer multiporzessor Maschine zum Crash führt...

Verwende eine CrticalSection um die Nutzung auf einen Thread zu einer Zeit zu begrenzen.

[cpp]
class CImageThreadSicher :
	public CImage
{
public:
	CImageThreadSicher(void);
public:
	~CImageThreadSicher(void);

private:
	CCriticalSection m_CS;
public:
	void SetPixel(int x,int y,COLORREF color);
	void Draw(HDC hDestDC,const RECT& rectDest);
};

void CImageThreadSicher::SetPixel(int x,int y,COLORREF color){
	m_CS.Lock();
	CImage::SetPixel(x,y,color);
	m_CS.Unlock();
}

Es scheint zu funktionieren