2024-04-07 事後報告

背景

著者注: 迅速な報告に感謝します。ノード運営者の方々には、異常を発見した場合はいつでもすぐにご連絡いただくようお願いいたします。

データの調査

• ブロック 3,191,088: アクティベート | SECRET_LOCK が作成されました (有効期限は 420 ブロック)
• ブロック 3,191,098: ディアクティベート | SECRET_LOCK が完了しました
• ブロック 3,191,525: アクティベート | SECRET_LOCK が作成されました (有効期限は 420 ブロック)
• ブロック 3,191,538: アクティベート | SECRET_LOCK が作成されました (有効期限は 420 ブロック)
• ブロック 3,191,576: ディアクティベート | SECRET_LOCK が完了しました

著者の注: この問題とは無関係ですが、シークレットの再利用は一般的に安全でないことを強調しておく必要があります。シンボルは (ほとんどのチェーンと同様に) メモリ使用量を削減するために、使用されるすべてのシークレットを追跡しません。したがって、シークレットロックを使用する際は、この点に注意してください。

1. ブロック 3,191,525 で SECRET_LOCK が作成されました。
2. ブロック 3,191,508 が最終化されました。
3. 一致するすべての SECRET_LOCK が削除されました。
4. ブロック 3,191,538 で SECRET_LOCK が作成されました。

解決方法

	auto groupIter = groupedSet.find(key);
	const auto* pGroup = groupIter.get();
	if (!pGroup)
		return;

	for (const auto& identifier : pGroup->identifiers())
		set.remove(identifier);

	groupedSet.remove(key);

	std::unordered_set<typename TDescriptor::KeyType, utils::ArrayHasher<typename TDescriptor::KeyType>> pendingRemovalIds;
	ForEachIdentifierWithGroup(*m_pDelta, *m_pHeightGroupingDelta, height, [height, &pendingRemovalIds](auto& history) {
		history.prune(height);

		if (history.empty())
			pendingRemovalIds.insert(history.id());
	});

	for (const auto& identifier : pendingRemovalIds)
		m_pDelta->remove(identifier);

	m_pHeightGroupingDelta->remove(height);

著者の注: このプロセスを通じてステートハッシュ検証が成功したのは興味深い点です。ブロック高 3,191,525 で作成された (2番目の) シークレットロックがキャッシュに追加され、ステートハッシュが変更されました。ブロック *526 と *537 の間のどこかで、シークレットロックはプルーニングされました。 プルーニングは、ステートに影響を与えることができなくなったエンティティのみを削除するため、ステートに影響を与えないと考えられています。ブロック *538 で作成された (3番目の) シークレットロックがキャッシュに追加され、ステートハッシュが変更されました。 実質的に、これはシークレットロックの更新としてキャプチャされました。

	if (height == lockInfo.EndHeight || forceHeights.cend() != forceHeights.find(height))
		accountStateConsumer(accountStateCache.find(lockInfo.OwnerAddress).get());