Changeset 2190 for XIOS/dev/dev_ym/XIOS_COUPLING/src/transformation/domain_algorithm/domain_algorithm_reorder.cpp

Timestamp:

07/20/21 16:22:31 (3 years ago)

Author:

jderouillat

Message:

Add bound checking when the workflow view is scaned (highlighted by blitz check in debug mode)

File:

• XIOS/dev/dev_ym/XIOS_COUPLING/src/transformation/domain_algorithm/domain_algorithm_reorder.cpp (modified) (1 diff)

XIOS/dev/dev_ym/XIOS_COUPLING/src/transformation/domain_algorithm/domain_algorithm_reorder.cpp

@@ -110 +110 @@
    int countMasked(0); // countMasked will store the offset index between full and workflow views
    for (size_t i = 0; i < indexSize ; ++i) {
-     if ( sourceFullIdx(i)==sourceWorkflowIdx(i-countMasked) ) {
-       domainDestination->mask_1d(i) = 1;
+       if ( ( (i-countMasked) >= sourceWorkflowIdx.numElements() )
+         || ( sourceFullIdx(i)!=sourceWorkflowIdx(i-countMasked) ) ) {
+       domainDestination->mask_1d(i) = 0;
+       countMasked++;
      }
      else {
-       domainDestination->mask_1d(i) = 0;
-       countMasked++;
+       domainDestination->mask_1d(i) = 1;
      }
    }