diff -r 022fc9a9f172 -r 43cbef41dd6d email/pop3andsmtpmtm/clientmtms/src/MIUTMSG.CPP
--- a/email/pop3andsmtpmtm/clientmtms/src/MIUTMSG.CPP	Sun Mar 28 16:36:08 2010 +0100
+++ b/email/pop3andsmtpmtm/clientmtms/src/MIUTMSG.CPP	Mon Mar 29 12:25:56 2010 +0100
@@ -6925,6 +6925,11 @@
 			attachment->SetSize(iSize);
 		file.Close();
 		TFileName filename = HtmlFilename(iMsvEntry, GetDefaultAttachmentName());
+		if(filename.Length() == 0)
+		    {
+		    User::Leave(KErrArgument);
+		    }
+
 		attachment->SetAttachmentNameL(filename);
 		CleanupStack::Pop(attachment); // ownership passed to attachment manager 
 		
@@ -6939,6 +6944,11 @@
 	TMsvEmailEntry emailEntry = iMsvEntry.Entry();
 	
 	TFileName filename = HtmlFilename(iMsvEntry, GetDefaultAttachmentName());
+	if(filename.Length() == 0)
+		{
+		User::Leave(KErrArgument);
+		}
+
 	emailEntry.iDetails.Set(filename);
 	emailEntry.iSize = iSize;	
 	return iMsvEntry.ChangeL(emailEntry, aStatus);	
@@ -6946,8 +6956,9 @@
 
 TFileName CImHtmlConverter::HtmlFilename(CMsvEntry& aEntry, TPtrC aFileName)
 	{
-	TFileName path;
 	TFileName name;
+	name.Zero();
+
 	TMsvEntry entry = aEntry.Entry();	
 // this is not used at all , need to get rid of this 
 	if (entry.iDetails.Length())