Fixed potential bug. Missing node adding on RecS item matching.

astand · astand · commit 02d1e28b40b4 · 2021-10-10T21:31:36.000+03:00
diff --git a/src/maincli.cpp b/src/maincli.cpp
@@ -109,12 +109,13 @@ int main(int argc, char* argv[])
 
         for (size_t recs = 0; recs < m->RecS.size(); recs++)
         {
-          // test all recs
-          found = (std::find(nodes.begin(), nodes.end(), nodename) != nodes.end());
+          std::string rx_node_name = m->RecS[recs];
 
-          if (!found)
+          // test all recs
+          if (std::find(nodes.begin(), nodes.end(), rx_node_name) == nodes.end())
           {
-            nodes.push_back(nodename);
+            // New node name, put it in the node collection
+            nodes.push_back(rx_node_name);
           }
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -109,12 +109,13 @@ int main(int argc, char* argv[])`
`109`	`109`
`110`	`110`	`for (size_t recs = 0; recs < m->RecS.size(); recs++)`
`111`	`111`	`{`
`112`		`- // test all recs`
`113`		`- found = (std::find(nodes.begin(), nodes.end(), nodename) != nodes.end());`
	`112`	`+ std::string rx_node_name = m->RecS[recs];`
`114`	`113`
`115`		`- if (!found)`
	`114`	`+ // test all recs`
	`115`	`+ if (std::find(nodes.begin(), nodes.end(), rx_node_name) == nodes.end())`
`116`	`116`	`{`
`117`		`- nodes.push_back(nodename);`
	`117`	`+ // New node name, put it in the node collection`
	`118`	`+ nodes.push_back(rx_node_name);`
`118`	`119`	`}`
`119`	`120`	`}`
`120`	`121`	`}`