crytic/slither

slither-read-storage: add support for nested structs

Open

#2,077 opened on 2023年8月3日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)Python (4,769 stars) (886 forks)batch import
enhancementgood first issue

説明

For example: https://github.com/OpenZeppelin/openzeppelin-contracts-upgradeable/blob/e6e6d73c6219a9e436cdc91ac4f943c7699831c0/contracts/utils/structs/EnumerableSetUpgradeable.sol#L233-L235

This needs to be updated to calculate the slot for UserDefinedType https://github.com/crytic/slither/blob/3f90e86badfa956b5cc692e0421c2b8065c31d43/slither/tools/read_storage/read_storage.py#L593

Test case slither-read-storage bsc:0x52Dcd80506bc421cae6927C73f306FD0d083e43a

コントリビューターガイド

技術スタック
python
領域
toolingcli
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
PythonSolidity storage layoutUnderstanding of slither read storage
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
The issue requires modifying the function at slither/tools/read storage/read storage.py line 593 to handle UserDefinedType for nested structs. The test case 'slither read storage bsc:0x52Dcd80506bc421cae6927C73f306FD0d083e43a' can be used to verify the change. Look at the linked code in openzeppelin contracts upgradeable for an example of nested struct usage. No maintainer clarification is needed.