Re: Sending e-mail from code

girishshetty — Fri, 24 Jun 2005 11:53:10 +0200

Some suggestion/comments on ur code:

This is ur ConstruL:

void CSendSynchMail::ConstructL()
{
   // connect to MTM server
   iSession = CMsvSession::OpenSyncL(*this);
   CleanupStack::PushL(iSession);

   // create MTM registry
   iMtmRegistry = CClientMtmRegistry::NewL(*iSession);
   CleanupStack::PushL(iMtmRegistry);

   // create smtp MTM
   iMtm = static_cast<CSmtpClientMtm*>(iMtmRegistry->NewMtmL(KUidMsgTypeSMTP));
   CleanupStack::PushL(iMtm);

   // find service
   iServiceId = FindServiceByTypeL(KUidMsgTypeSMTP);
   if( iServiceId==KMsvRootIndexEntryId )
      User::Leave(KErrNotFound);

   // Ui registry
   iUiRegistry = CMtmUiRegistry::NewL(*iSession);
   CleanupStack::PushL(iUiRegistry);
   iMtmUi = iUiRegistry->NewMtmUiL( *iMtm );

   CleanupStack::Pop(4);
}

Never ever write such code in Program as,

ur destructer is like this:

CSendSynchMail::~CSendSynchMail()
{
   delete iMtmUi;
   delete iUiRegistry;
   delete iMtm;
   delete iMtmRegistry;
   delete iSession;
}

Suppose that, ur ConstructL leaves in between. lets say some where after after "CleanupStack::PushL(iSession);"
So, TRAP down the line will take care of deleting everything those items that are in CleanupStack, and obviously ur iSession is one among them.
Now in ur destructer, again u r deleting iSession, in which case, you are doing double deletion and your appln will PANIC.

So, never Push Members Variables to Cleanup Stack, if you are deleting them in destructer.

Regards
Girish

Sending e-mail from code

LtSolo — Thu, 28 Apr 2005 08:20:59 +0200

NewLC - Sending e-mail from code - Comments

Re: Sending e-mail from code

Sending e-mail from code