applicationmanagement/omascomoadapter/src/omascomoadapter.cpp
changeset 67 fdbfe0a95492
parent 47 d316aecb87fd
--- a/applicationmanagement/omascomoadapter/src/omascomoadapter.cpp	Fri Sep 17 19:58:05 2010 +0530
+++ b/applicationmanagement/omascomoadapter/src/omascomoadapter.cpp	Fri Oct 15 11:39:02 2010 +0530
@@ -977,9 +977,10 @@
             
 
     		CNSmlDMAlertItem* item = new (ELeave) CNSmlDMAlertItem ;
-    		
+    		CleanupStack::PushL(item);
     		HBufC8 *data = HBufC8::NewL(1024);
-            
+        CleanupStack::PushL(data);    
+        	
             TPtr8 ptr = data->Des();
             //ptr.Append(KDataStart);
             ptr.Append(KResultCodeStart);
@@ -990,48 +991,43 @@
             ptr.Append(KIdentifierEnd);
             //ptr.Append(KDataEnd);
 
-            HBufC8 *sourceuri = genericalerts[i]->iSourceURI;
+        HBufC8 *sourceuri = genericalerts[i]->iSourceURI;
     		
     		item->iSource = sourceuri;
 
-			HBufC8 *targeturi = targetURIGet.AllocL();
-    		
+				HBufC8 *targeturi = targetURIGet.AllocL();
+    		CleanupStack::PushL(targeturi);  
     		item->iTarget = targeturi;
     		
     		HBufC8 *metatype = scomometatype.AllocL();
-    		            
+    		CleanupStack::PushL(metatype);            
     		item->iMetaType = metatype;
     		            
-    		HBufC8 *metaformat = scomometaformat.AllocL();         
-
+    		HBufC8 *metaformat = scomometaformat.AllocL();
+    		CleanupStack::PushL(metaformat);
     		item->iMetaFormat = metaformat;
 
     		HBufC8 *metamark = scomomark.AllocL();
-    		            
+    		CleanupStack::PushL(metamark);            
     		item->iMetaMark = metamark;
     
     		item->iData = data;
     
     		iItemArray.AppendL(*item);
             
-            TRAP_IGNORE(privateAPI.AddDMGenericAlertRequestL(              
-                    *genericalerts[i]->iCorrelator,iItemArray ));
-                    
-                    
-            delete data;
-            
-            delete metamark;
+        TRAP_IGNORE(privateAPI.AddDMGenericAlertRequestL(              
+                    *genericalerts[i]->iCorrelator,iItemArray ));       
             
-            delete metaformat;
-            
-            delete metatype;
-            
-            delete targeturi;
-            delete item;
+        CleanupStack::PopAndDestroy( metamark);            
+        CleanupStack::PopAndDestroy( metaformat);           
+        CleanupStack::PopAndDestroy( metatype);	            
+        CleanupStack::PopAndDestroy( targeturi);	
+        CleanupStack::PopAndDestroy( data);	
+        CleanupStack::PopAndDestroy( item);
                     
-            iItemArray.Reset();			
-			iItemArray.Close();
-            CleanupStack::PopAndDestroy( &privateAPI);
+        iItemArray.Reset();			
+				iItemArray.Close();
+        CleanupStack::PopAndDestroy( &privateAPI);
             }
         }
     genericalerts.ResetAndDestroy();
@@ -2598,7 +2594,7 @@
                                     else
                                         if (dataType == EDCConRef)
                                             {
-                                            CBufBase *b = CBufFlat::NewL(4);
+                                            CBufBase *b = CBufFlat::NewL(12);
                                             CleanupStack::PushL(b);
                                             SessionL().DeploymentComponentDataL(
                                                     iluid, dataType, *b);
@@ -4144,7 +4140,7 @@
         RDEBUG( "CSCOMOAdapter::CheckStateChangesL(): Adding Trust Closing Session" );
         iManagement.Close();
         iSessionOpened = EFalse;
-        RApplicationManagement &session = SessionL();
+//        RApplicationManagement &session = SessionL(); // coverity Fix
         RDEBUG( "CSCOMOAdapter::CheckStateChangesL(): Adding Trust new session started" );
         }
     }