cryptoadvance / cryptoadvance/specter-diy

Memory allocation problems with multiple wallets

Open
#260 1 comment 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
585
Forks
108
Avg merge
6d 1h
Merged PRs (30d)
5

Description

Steps to reproduce:
- create 10 multisig wallets (i.e. 3 of 6)
- enjoy the problems with memory allocation

Expected behaviour:
- should work with arbitrary number of wallets

Solution:

Don't keep all wallets in RAM (only keep names), load them as they are needed. This will slow down PSBT parsing, but will work with any number of wallets. To speed up things maybe makes sense to off-load them to SDRAM instead of MCU internal RAM.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reproducing the problem with 10 multisig wallets and observe the memory allocation failure. Trace how wallets are retained in RAM and how PSBT parsing loads them, then verify that wallets can be created and used in arbitrary numbers without exhausting MCU memory.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
embedded-iot
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.