0023265: cppcheck warning: Mismatching allocation and deallocation

Use pairs: new - delete (new - delete [] for arrays), malloc - free

diff --git a/src/QADNaming/QADNaming_IteratorsCommands.cxx b/src/QADNaming/QADNaming_IteratorsCommands.cxx
index 4416333648fa71d03cb06f22a5a9c3e139211b0b..aa4bcd71e68bcfa4fd32aa3a1f5636f8d3d2a1d7 100755 (executable)
--- a/src/QADNaming/QADNaming_IteratorsCommands.cxx
+++ b/src/QADNaming/QADNaming_IteratorsCommands.cxx
@@ -158,7 +158,7 @@ static Standard_Integer GetAllNewShapes (Draw_Interpretor& di,
            sprintf(aSubName,"%s_%d",aName,a);
          } else aSubName = NULL;
          aResult+=GetAllNew(anIter.NewShape(),aLabel,aSubName,0);
-         if (aSubName != NULL) delete(aSubName);
+         if (aSubName != NULL) delete [] aSubName;
        }
       }
     } else { // shape

diff --git a/src/QANewModTopOpe/QANewModTopOpe.cxx b/src/QANewModTopOpe/QANewModTopOpe.cxx
index 7304b80b0969f6527b6e78b47d84ed53b09df28d..86a190cc45385e4f0ca9721840027509106d1d3a 100755 (executable)
--- a/src/QANewModTopOpe/QANewModTopOpe.cxx
+++ b/src/QANewModTopOpe/QANewModTopOpe.cxx
@@ -659,7 +659,7 @@ Standard_Boolean QANewModTopOpe::IsConnected(const TopoDS_Shape& TheS)
     if(aNotChanged) break;
     if(aRes) break;
   }
-  delete aMat;
+  delete [] aMat;
 
   //  cout << "QANewModTopOpe::IsConnected END: aRes="<<aRes   << ";ncount="<<ncount<<endl;
   cout << "QANewModTopOpe::IsConnected END: aRes="<<aRes <<endl;

diff --git a/src/TestTopOpeTools/TestTopOpeTools_TraceCommands.cxx b/src/TestTopOpeTools/TestTopOpeTools_TraceCommands.cxx
index 3dde2462605927f3b3b3818d205cc21a73d4ae25..fed9bc0086094b8e4d5833a004cc2eeb5f9de629 100755 (executable)
--- a/src/TestTopOpeTools/TestTopOpeTools_TraceCommands.cxx
+++ b/src/TestTopOpeTools/TestTopOpeTools_TraceCommands.cxx
@@ -319,7 +319,7 @@ Standard_Integer dstrace(Draw_Interpretor& di, Standard_Integer n , const char**
   pa[1] = (char *)a1.ToCString();
   for (Standard_Integer i=1;i<n;i++) pa[i+1] = a[i];
   TestTopOpeTools_SetTrace(di,npa,pa);
-  if (pa) delete [] pa;
+  if (pa) free(pa);
   return 0;
 }