0xMiden / 0xMiden/protocol

Limit number of storage maps per account to 64

Aberta
#2,996 1 comentário 0 reações 0 responsáveis Ver no GitHub
kernels rust
Linguagem predominante
Rust
Estrelas
132
Forks
167
Merge médio
1d 23h
PRs com merge (30d)
110

Descrição

As motivated in https://github.com/0xMiden/node/issues/2121#issuecomment-4527244793, we should limit the number of storage maps per account to 64.

For now, storage can only be set at account creation time, so we only need to assert this then. We can compute the number of storage maps in `$kernel::account::insert_new_storage` and assert it is <= 64.

On the Rust side, we should also validate this in `AccountStorage`. We can add a constant:

```rust
impl AccountStorage {
pub const MAX_NUM_STORAGE_MAPS: usize = 64;
}
```

And then add validation in `AccountStorage::new`.

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.