How to fix COMPtr' problem?

[D_D_K]

Hi,

After running the following code, my cad crashes at the end.

because there are some custom functions with parameter IAcadBlocksPtr, so I used it like below: 

CComQIPtr<IAcadApplication> pApp(acedGetIDispatch(FALSE));
CComPtr<IAcadDocument> pDoc;  //getcurrent doc

IAcadBlocksPtr pblks;
hR = pDoc->get_Blocks(pblks); 

//do something Ok

pblks.Release();

HRESULT hr = pDoc->Close(vb); //Ok
pDoc.Release();

return true;// <---crash;

How do I avoid this?

[Alexander.Rivilis]

1. pDoc is not initilaize

2. pDoc release twice as far as you call pDoc.Release() and destructor of pDoc also call pDoc.Release()

[D_D_K]

Hi sir,

the reason I call pDoc.release(); because when using the original code below -> cad crashes first.

I'm not sure where is wrong. these variables are initialized and working correctly.

CComQIPtr<IAcadApplication> pApp(acedGetIDispatch(FALSE));
CComPtr<IAcadDocument> pDoc;  
pApp->get_ActiveDocument(&pDoc);

IAcadBlocksPtr pblks;
hR = pDoc->get_Blocks(pblks); 

//do something Ok

pblks.Release();
HRESULT hr = pDoc->Close(vb); //success

return true;// <---crash;

[Alexander.Rivilis · Accepted solution]

Are you trying to close active document?

It is look like error not in this code.

[D_D_K]

Thanks sir, 

I have reviewed my code. the error comes from a subroutine.

Remove it and now the code works fine.