crytic/slither

slither-read-storage: add support for nested structs

Open

#2.077 aberto em 3 de ago. de 2023

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Python (886 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (4.769 stars)
Métricas de merge de PR
 (Mesclagem média 54d 9h) (2 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine a lógica de cálculo de slot de armazenamento em read storage.py por volta da linha 593 e estenda a para lidar com tipos struct aninhados calculando recursivamente o deslocamento do slot para cada campo aninhado.
Pilha de tecnologia
python
Domain
blockchain
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonSolidity basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50