mpserviceplugins/mpxsqlitedbcommon/src/mpxdbmanager.cpp
changeset 48 af3740e3753f
parent 42 79c49924ae23
--- a/mpserviceplugins/mpxsqlitedbcommon/src/mpxdbmanager.cpp	Tue Jul 06 14:13:36 2010 +0300
+++ b/mpserviceplugins/mpxsqlitedbcommon/src/mpxdbmanager.cpp	Wed Aug 18 09:46:20 2010 +0300
@@ -505,7 +505,7 @@
     CleanupClosePushL( dstFile );
     
     // resize destination file
-    TInt remainingBytes;
+    TInt remainingBytes = 0;
     User::LeaveIfError( srcFile.Size(remainingBytes) );
     User::LeaveIfError( dstFile.SetSize(remainingBytes) );
 
@@ -1854,6 +1854,8 @@
             {
             delete database.iAliasname;
             database.iAliasname = HBufC::NewL(KAliasName().Length());
+            // coverity[size_error]
+            // coverity[buffer_alloc]
             HBufC* temp = HBufC::NewLC(2); // form of DE, DF, DX,...
             temp->Des().Append(iRAMDrive); // length == 2
             TDriveUnit pdrive( database.iDrive );
@@ -2208,6 +2210,8 @@
     {
     TInt columnCount(GetColumnCountL(aStatement));
     TInt err(KErrNone);
+    // coverity[incorrect_multiplication]
+    // coverity[buffer_alloc]
     HBufC* tableRow = HBufC::NewLC(255 * columnCount);
     TPtr tableRowPtr = tableRow->Des();
 
@@ -2299,6 +2303,7 @@
     const TDesC& aAlias,
     RArray<HBufC*>& aTableName)
     {
+    CleanupClosePushL(aTableName);
     RSqlStatement statement;
     CleanupClosePushL(statement);
 
@@ -2329,6 +2334,7 @@
         }
 
     CleanupStack::PopAndDestroy(&statement);
+    CleanupStack::Pop();
     }
 
 // ----------------------------------------------------------------------------