]> SALOME platform Git repositories - tools/medcoupling.git/commitdiff
Salome HOME
[OverlapDEC] valgrind on uninitialized array of bool
authorabn <adrien.bruneton@cea.fr>
Thu, 21 Dec 2023 09:11:24 +0000 (10:11 +0100)
committerabn <adrien.bruneton@cea.fr>
Mon, 22 Jan 2024 12:31:36 +0000 (13:31 +0100)
src/ParaMEDMEM/OverlapMapping.cxx

index 203dd3ff3b2a5dbf71f7cff1ed0a72dbf82cb2c0..b19d1706e1c1f9b8c1bc553918e05b705103ba60 100644 (file)
@@ -619,7 +619,9 @@ void OverlapMapping::multiply(const MEDCouplingFieldDouble *fieldInput, MEDCoupl
   INTERP_KERNEL::AutoPtr<double> tmp=new double[nbOfCompo];
 
   // By default field value set to default value - so mark which cells are hit
-  INTERP_KERNEL::AutoPtr<bool> hit_cells = new bool[fieldOutput->getNumberOfTuples()];
+  mcIdType ntup = fieldOutput->getNumberOfTuples();
+  INTERP_KERNEL::AutoPtr<bool> hit_cells = new bool[ntup];
+  std::fill((bool *)hit_cells, (bool *)hit_cells+ntup, false);
 
   for(vector<int>::const_iterator itProc=_the_matrix_st_source_proc_id.begin(); itProc != _the_matrix_st_source_proc_id.end();itProc++)
   // For each source processor corresponding to a locally held matrix:
@@ -750,7 +752,7 @@ void OverlapMapping::multiply(const MEDCouplingFieldDouble *fieldInput, MEDCoupl
 
   // Fill in default values for cells which haven't been hit:
   int i = 0;
-  for(bool * hit_cells_ptr=hit_cells; i< fieldOutput->getNumberOfTuples(); hit_cells_ptr++,i++)
+  for(bool * hit_cells_ptr=hit_cells; i< ntup; hit_cells_ptr++,i++)
     if (!(*hit_cells_ptr))
       {
         double * targetPt=fieldOutput->getArray()->getPointer();