When initially parsing and loading blocks we attempt to validate their parsed attributes and apply auto-fixes when invalid, before attempting to validate again after the fixes have been applied. This is done after we get a new raw block back after a round of fix attempts, which we can revalidate.

However, looking at this section of code it appears as though we are revalidating the original unvalidatedBlock instead of fixedBlock during the second execution of validateBlock().

It looks to me like on the second run of validateBlock(), we should be passing fixedBlock as the first param instead.