From 061e65be9386b751b6e5d41bea8c390eda460c8e Mon Sep 17 00:00:00 2001 From: Ori Newman Date: Wed, 9 Mar 2022 21:11:00 +0200 Subject: [PATCH] Fix argument order for IsAncestorOf in boundedMergeBreakingParents (#1966) --- .../processes/consensusstatemanager/pick_virtual_parents.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/domain/consensus/processes/consensusstatemanager/pick_virtual_parents.go b/domain/consensus/processes/consensusstatemanager/pick_virtual_parents.go index a232a88d6..78b15e273 100644 --- a/domain/consensus/processes/consensusstatemanager/pick_virtual_parents.go +++ b/domain/consensus/processes/consensusstatemanager/pick_virtual_parents.go @@ -336,12 +336,12 @@ func (csm *consensusStateManager) boundedMergeBreakingParents(stagingArea *model log.Debugf("Checking whether parent %s breaks the bounded merge set", parent) isBadRedInPast := false for _, badRedBlock := range badReds { - isBadRedInPast, err = csm.dagTopologyManager.IsAncestorOf(stagingArea, parent, badRedBlock) + isBadRedInPast, err = csm.dagTopologyManager.IsAncestorOf(stagingArea, badRedBlock, parent) if err != nil { return nil, err } if isBadRedInPast { - log.Debugf("Parent %s is an ancestor of bad red %s", parent, badRedBlock) + log.Debugf("Parent %s is a descendant of bad red %s", parent, badRedBlock) break } }