Changeset ad0929

Timestamp:

Mar 4, 2024, 9:34:12 PM (20 months ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.0, stable

Children:

9782e20

Parents:

02f14d

Message:

FIX: BondGraph::GetBondLength() accessed off by one.

File:

• src/Graph/BondGraph.cpp (modified) (1 diff)

src/Graph/BondGraph.cpp

@@ -125 +125 @@
     int secondZ) const
 {
+  int firstIndex = firstZ-1;
+  int secondIndex = secondZ-1;
   double return_length;
-  if ((firstZ < 0) || (firstZ >= (int)BondLengthMatrix->Matrix[0].size()))
+  if ((firstIndex < 0) || (firstIndex >= (int)BondLengthMatrix->Matrix[0].size()))
     return -1.;
-  if ((secondZ < 0) || (secondZ >= (int)BondLengthMatrix->Matrix[0][firstZ].size()))
+  if ((secondIndex < 0) || (secondIndex >= (int)BondLengthMatrix->Matrix[0][firstIndex].size()))
     return -1.;
   if (BondLengthMatrix == NULL) {
     return_length = -1.;
   } else {
-    return_length = BondLengthMatrix->Matrix[0][firstZ][secondZ];
+    return_length = BondLengthMatrix->Matrix[0][firstIndex][secondIndex];
   }
   LOG(4, "INFO: Request for length between " << firstZ << " and "