crytic/slither

slither-read-storage: add support for nested structs

Open

#2 077 ouverte le 3 août 2023

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Python (886 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (4 769 stars)
Métriques de merge PR
 (Merge moyen 54j 9h) (2 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez la logique de calcul de l'emplacement de stockage dans read storage.py autour de la ligne 593, puis étendez la pour gérer les types de structure imbriqués en calculant récursivement le décalage d'emplacement pour chaque champ imbriqué.
Stack technique
python
Domaine
blockchain
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PythonSolidity basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50