messagingappbase/mce/src/MceMessageListView.cpp
changeset 5 4697dfb2d7ad
parent 2 0bf1d54f37d9
child 9 1d7827e39b52
--- a/messagingappbase/mce/src/MceMessageListView.cpp	Tue Jan 26 11:55:08 2010 +0200
+++ b/messagingappbase/mce/src/MceMessageListView.cpp	Tue Feb 02 00:08:43 2010 +0200
@@ -1517,12 +1517,16 @@
                 if ( tEntry.iMtm == KUidMsgMMSNotification ) 
                     {
                     mmsCount++;
-                    if ( uiData->OperationSupportedL( KMtmUiFunctionDeleteMessage, tEntry ) )
+                    //Coverty fix,NULL RETURN, http://ousrv057/cov.cgi?cid=101801 
+                    if ( uiData )
                         {
-                        //MMS notification delete this entry not supported
-                        mmsNotDeleteCount ++;
+                        if ( uiData->OperationSupportedL( KMtmUiFunctionDeleteMessage, tEntry ) )
+                            {
+                            //MMS notification delete this entry not supported
+                            mmsNotDeleteCount ++;
+                            }
                         }
-                    }
+                     }
                 }
             }
         if ( mmsCount == count )
@@ -1824,10 +1828,14 @@
                 aMenuPane->SetItemDimmed( EMceCmdMove, ETrue );
                 //check also open               
                 CBaseMtmUiData* uiData = iMceUi->GetMtmUiDataL( KUidMsgMMSNotification );
-                aMenuPane->SetItemDimmed( EAknCmdOpen, 
-                    uiData->OperationSupportedL( KMtmUiFunctionOpenMessage, currentEntry ) );
-                aMenuPane->SetItemDimmed( EMceCmdDelete, 
-                    uiData->OperationSupportedL( KMtmUiFunctionDeleteMessage, currentEntry ) );               
+                // Coverty change, NULL Return, http://ousrv057/cov.cgi?cid=101800
+                if ( uiData )
+                    {
+                    aMenuPane->SetItemDimmed( EAknCmdOpen, 
+                     uiData->OperationSupportedL( KMtmUiFunctionOpenMessage, currentEntry ) );
+                    aMenuPane->SetItemDimmed( EMceCmdDelete, 
+                     uiData->OperationSupportedL( KMtmUiFunctionDeleteMessage, currentEntry ) );
+                    }                 
                 }
 
             aMenuPane->SetItemDimmed( EMceCmdOutboxSuspend, ETrue );
@@ -3075,7 +3083,11 @@
                 ( currentEntry.iMtm == KUidMsgMMSNotification ) )
                 {
                 CBaseMtmUiData* uiData = iMceUi->GetMtmUiDataL( KUidMsgMMSNotification );
-                skip = uiData->OperationSupportedL( KMtmUiFunctionOpenMessage, currentEntry );
+                // Coverty change, NULL Return, http://ousrv057/cov.cgi?cid=101803
+                if ( uiData )
+                    {
+                	skip = uiData->OperationSupportedL( KMtmUiFunctionOpenMessage, currentEntry );
+                	}	
                 }
             }
         }    
@@ -3803,7 +3815,8 @@
                     iSession->GetEntry( id, serviceId, currentEntry );
                            
                     CBaseMtmUiData* mtmUiData = iMceUi->GetMtmUiDataL( currentEntry.iMtm );
-                    if ( currentEntry.iMtm == KUidMsgMMSNotification &&
+                    // Coverty fix NULL Return, http://ousrv057/cov.cgi?cid=101802
+                    if ( currentEntry.iMtm == KUidMsgMMSNotification && mtmUiData &&
                         mtmUiData->OperationSupportedL(KMtmUiFunctionOpenMessage, currentEntry ) )
                         {
                         // when MMSNotification is in retrieving status, the hide open for MSK
@@ -3812,7 +3825,7 @@
                     else 
                         {
                         resourceId = R_MCE_MSK_BUTTON_OPEN;
-                        }   
+                        } 
                     }
                 else
                     {
@@ -3879,7 +3892,8 @@
                     iSession->GetEntry( id, serviceId, currentEntry );
                            
                     CBaseMtmUiData* mtmUiData = iMceUi->GetMtmUiDataL( currentEntry.iMtm );
-                    if ( currentEntry.iMtm == KUidMsgMMSNotification &&
+                    // Coverty fix, NULL Return, http://ousrv057/cov.cgi?cid=101802
+                    if ( currentEntry.iMtm == KUidMsgMMSNotification && mtmUiData &&
                         mtmUiData->OperationSupportedL(KMtmUiFunctionOpenMessage, currentEntry ) )
                         {
                         // when MMSNotification is in retrieving status, the hide open for MSK
@@ -3888,7 +3902,7 @@
                     else 
                         {
                         resourceId = R_MCE_MSK_BUTTON_OPEN;
-                        }   
+                        } 
                     }
                 else
                     {