cryptoadvance / cryptoadvance/specter-diy

QR Scanner Sometimes Returns an Error from Sparrow

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

Description

`Can't find matching app for this request:`
![IMG_7926](https://github.com/cryptoadvance/specter-diy/assets/448644/5f934153-b010-489e-8ef8-ceeb3cfc536a)

I discovered this in a strange way. First I was doing a CPFP transaction and got this error, followed by a long binary string. I tried RBF instead and Specter-DIY was able to sign without issues.

Then today I tried a new transaction that is a 1 input 1 output `p2wsh` transaction and I got the error again. Sometimes the error is an empty string, sometimes it's a long binary string. At first it seemed like selecting `decrease density` on Sparrow caused an empty string, but now it's happening both ways.

Or sometimes it returns this memory allocation error (I deleted the other wallets on this device and restarted to see if that made a difference):
![IMG_7927](https://github.com/cryptoadvance/specter-diy/assets/448644/cf6ca158-29b4-47ad-893f-ce1a34e779e1)

IIRC I only get the memory allocation error when using decreased density.

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the QR scanner failure from Sparrow using CPFP, RBF, and the 1-input/1-output p2wsh transaction, testing both density settings. Compare the empty, binary-string, and memory-allocation failures and trace the QR decoding and memory-allocation paths. Done means these transaction types scan reliably without errors.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
embedded-iot, security
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.