Author: Gavin Andresen 2011-09-27 20:39:32
Published on: 2011-09-27T20:39:32+00:00
The code in question is a "belt-and-suspenders just in case" case that should be impossible to trigger. AcceptBlock() is called from two places in the code: ProcessBlock, if the block is not an orphan, and recursively, in AcceptBlock(), processing orphans that link to the block being accepted. The mapBlockIndex.find(hashPrevBlock) in AcceptBlock can't fail. In case there is some subtle bug or edge case, keeping the check is appropriate. Attackers look for subtle bugs and edge cases that can be exploited.
Updated on: 2023-06-04T20:05:27.388686+00:00