solana-labs/solana

Missing invoke translation test

Open

#13 955 ouverte le 4 déc. 2020

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Rust (3 950 forks)batch import
good first issueruntime

Métriques du dépôt

Stars
 (12 651 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Problem

Missing a test that verifies the AccountInfo's data length is correctly enforced to exist in RW data. The length is translated correctly but we have no way to ensure that it will continue to do so over time.

https://github.com/solana-labs/solana/blob/5dceddd21d3c1cf874b003a0c25b1f21f4926614/programs/bpf/rust/ro_modify/src/lib.rs#L158

Proposed Solution

AccountInfo's data is expressed as an Rc<RefCell<&mut [u8]>>, need a way to construct this where the slice's len is in RO memory (like the RO segment of a BPF ELF) while the data pointer points to RW memory. This is not a use case that can be activated without malicious intent.

Guide contributeur

Direction de recherche
Étudiez comment construire un Rc<RefCell<&mut [u8]>> où la longueur du slice est dans la mémoire en lecture seule (par exemple, le segment RO d'un ELF BPF) tandis que le pointeur de données pointe vers la mémoire RW. Regardez les modèles de test existants dans les tests de programme BPF de Solana.
Stack technique
rust
Domaine
blockchain
Type d'issue
Test
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.
Une demi journée
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.
Investigation requise
Prérequis
RustSolana
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50