Help! I cannot destory CMsvEntry object.

[autumnsea]

I am using CMsvEntry and encountering some problem.


CMsvEntry * iMsvEntry = CMsvEntry::NewL(*iSession, KMsvGlobalOutBoxIndexEntryId, TMsvSelectionOrdering());

CleanupStack::PushL(iMsvEntry);

//if I put CleanupStack::PopAndDestroy(iMsvEntry); nothing wrong

iMsvEntry=iSession->GetEntryL(aId);

CleanupStack::PopAndDestroy(iMsvEntry);//But if I put the code here an

Access Violation occurs, could any please tell me why? How I can destroy
this object? I've been looking for solutions for a day. Thank you

[wizard_hu_]

CMsvEntry * iMsvEntry = CMsvEntry::NewL(*iSession, KMsvGlobalOutBoxIndexEntryId, TMsvSelectionOrdering());
CleanupStack::PushL(iMsvEntry);
iMsvEntry=iSession->GetEntryL(aId);
CleanupStack::PopAndDestroy(iMsvEntry);

In the last line, you verify, if top of the Cleanup Stack contains the pointer returned by iSession->GetEntryL(aId) (since iMsvEntry contains this value). However the topmost entry of the Cleanup Stack contains a pointer referring the object created by CMsvEntry::NewL(...). Since the two (expected and actual) are not equal, you get the panic.

[autumnsea]

Oh thank you for the kind reply . So should I drop either of the two lines?
But what if the code does not work?

[wizard_hu_]

So should I drop either of the two lines?

I do not know, it highly depends on what you want to achieve.

But what if the code does not work?

Fix it, this is the common way

[Paul.Todd]

The problem is not with the cleanupstack, its with your c++..


You create a new instance of a CMsvEntry and put it on the cleanupstack

you then replace it with the value returned from GetEntry

You then remove it from the cleanup stack but use the value from GetEntry and not the one you allocated with NewL

CMsvEntry* aaaaa = CMsvEntry::NewL(*iSession, KMsvGlobalOutBoxIndexEntryId, TMsvSelectionOrdering());
CleanupStack::PushL(aaaaa);
CMsvEntry* iMsvEntry=iSession->GetEntryL(aId);
CleanupStack::PopAndDestroy(aaaaa);

[autumnsea]

Thank you for the timely reply

I know what the problem is now. I am still not very familiar with the SMS yet, maybe C++ inclusive

But here is another problem:

I wrote some code (as the 'SMS Example' and the 'SMSSend' example) to send SMSes automaticly, but once the problem has sent two or more messages, when closing the program there will be memory leakage. I will cannot figure out what is wong. If I remove the InvokeAsynFunctionL() from the SendL(), I can call SendL() twice with out problem. Is there something I need to pay attention to on this function?

Thank you.

Enrique