supl/locationomasuplprotocolhandler/protocolhandlerver2/src/epos_comasuplsession2.cpp
changeset 40 16118e24b93b
parent 25 73f6c2762ffe
--- a/supl/locationomasuplprotocolhandler/protocolhandlerver2/src/epos_comasuplsession2.cpp	Tue Jul 06 14:44:11 2010 +0300
+++ b/supl/locationomasuplprotocolhandler/protocolhandlerver2/src/epos_comasuplsession2.cpp	Wed Aug 18 10:11:45 2010 +0300
@@ -187,7 +187,7 @@
 
      if( aIMSI.Length() )
          {
-         iIMSI.Create( aIMSI );    
+         User::LeaveIfError(iIMSI.Create( aIMSI ));    
          }
 
      if(iPosHandler)
@@ -1719,6 +1719,7 @@
     TOMASuplUtcTime UtcTime;
     TOMASuplPositionEstimate PosEstimate;
     TDateTime TimeStamp;
+    //coverity[var_decl]
     TInt ZoneCode,Zone,altitude,AltitudeUncertainty, HorizontalAccuracy;
     TOMASuplAltitudeInfo AltitudeInfo;
     TInt latitude,longitude;
@@ -1775,6 +1776,7 @@
     msg.AppendNum(altitude);
     iTrace->Trace(msg,KTraceFileName, __LINE__); 
     msg.Copy(_L("Altitude uncertainty "));
+    //coverity[uninit_use_in_call] 
     msg.AppendNum(AltitudeUncertainty);
     iTrace->Trace(msg,KTraceFileName, __LINE__); 
 
@@ -1903,6 +1905,7 @@
     TOMASuplUtcTime UtcTime;
     TOMASuplPositionEstimate PosEstimate;
     TDateTime TimeStamp;
+    //coverity[var_decl]
     TInt ZoneCode,Zone,altitude,AltitudeUncertainty, HorizontalAccuracy;
     TOMASuplAltitudeInfo AltitudeInfo;
     TInt latitude,longitude;
@@ -1928,6 +1931,7 @@
     TReal pwrMjr, pwrMnr, pwrAlt;
             Math::Pow(pwrMjr, 1.1, UncertaintySemiMajor);
             Math::Pow(pwrMnr, 1.1, UncertaintySemiMinor);
+    //coverity[uninit_use] 
     Math::Pow(pwrAlt, 1.070000, AltitudeUncertainty); 
     UncertaintySemiMajorReal = 10 * (pwrMjr -1);
     UncertaintySemiMinorReal = 10 * (pwrMnr -1);
@@ -5480,11 +5484,15 @@
 
 void COMASuplSession::GetPeriodicTriggerParams()
     {
-    iTrace->Trace(_L("COMASuplSession::GetPeriodicTriggerParams"), KTraceFileName, __LINE__); 
-    iTriggerParams.Get(iNumFixes, iInterval, iStartTime);
-    iTotalNoOfFixes = iNumFixes;
-    iStartTimeCopy = iStartTime;
-    }        
+    iTrace->Trace(_L("COMASuplSession::GetPeriodicTriggerParams"),
+            KTraceFileName, __LINE__);
+    TInt ret = iTriggerParams.Get(iNumFixes, iInterval, iStartTime);
+    if (KErrNone == ret)
+        {
+        iTotalNoOfFixes = iNumFixes;
+        iStartTimeCopy = iStartTime;
+        }
+    }
 
 void COMASuplSession::StartPeriodicTriggerTimer()
     {
@@ -5687,7 +5695,7 @@
 				//Inform UI that ,trigger session has started...
 		InsertActiveSessionL();
 		TInt privacyId;
-		iNetworkPrivacy->NotifyLocationRequestL(*iSuplReqInfo, privacyId); 
+		TRAP_IGNORE(iNetworkPrivacy->NotifyLocationRequestL(*iSuplReqInfo, privacyId); )
 		iTrace->Trace(_L(" Calling InsertActiveSessionL"), KTraceFileName, __LINE__);
          
 	}
@@ -5726,7 +5734,7 @@
 				
 				
 				
-    	iNetworkPrivacy->NotifyLocationRequestL(*iSuplReqInfo, iRequestorId);
+    	TRAP_IGNORE(iNetworkPrivacy->NotifyLocationRequestL(*iSuplReqInfo, iRequestorId);)
     	
     }