hhanh00 / zwallet

Zcash and Ycash light shielded wallet
https://ywallet.app
MIT License
42 stars 20 forks source link

The AppImage version of Linux desktop wallet crashing when sending T2T #114

Closed skironDotNet closed 10 months ago

skironDotNet commented 11 months ago

Subject of the issue

the app is crashing after hit final send button

Your environment

Distributor ID: Linuxmint Description: Linux Mint 21 Release: 21 Codename: vanessa GU: XFCE

Steps to reproduce

Exclude all shield notes, go to QR T address, click Send arrow button, enter T address, follow with steps to send

I don't know if this relates to XFCE, or to Mint, or to AppImage or missing packages, I don't know if YWallet provides any logs I will keep posted, maybe I'll try in Mint Cinnamon and Ubuntu

skironDotNet commented 11 months ago

Same on Mint Cinnamon and Ubuntu. On Ubuntu executed the wallet from Terminal, and here are the log

thread '' panicked at 'called Result::unwrap() on an Err value: InvalidSecretKey', native/zcash-sync/src/note_selection/builder.rs:305:46 note: run with RUST_BACKTRACE=1 environment variable to display a backtrace fatal runtime error: failed to initiate panic, error 5 Aborted (core dumped)

skironDotNet commented 11 months ago

Confirmed works on Android

skironDotNet commented 11 months ago

Also crash when trying to send all shield notes to shield address,

thread '' panicked at 'called Result::unwrap() on an Err value: InvalidSecretKey', native/zcash-sync/src/note_selection/builder.rs:305:46 note: run with RUST_BACKTRACE=1 environment variable to display a backtrace fatal runtime error: failed to initiate panic, error 5 Aborted (core dumped)

hhanh00 commented 11 months ago

See #111 #104

skironDotNet commented 11 months ago

for sure not #111 because I've read all the issue and were paying attention but #104 I need to test

hhanh00 commented 11 months ago

Have you changed the derivation path of the taddr?

skironDotNet commented 11 months ago

Yes, but the other test was from imported sapling private key only and also crash. I did disable the sound option as mentioned in #104 and didn't help. The error is solid meaning it does not imply the sound,

thread '' panicked at 'called Result::unwrap() on an Err value: InvalidSecretKey', native/zcash-sync/src/note_selection/builder.rs:305:46

On Windows all works whether changed derivation and doing T2T or imported sapling priv key only account and sending from that sapling. Linux is doomed for send, Android 12 is doomed for connect (but I don't care about Android).

Thank you for your time

hhanh00 commented 11 months ago

If you import a sapling key, you cannot make t2t txs. There is no seed phrase and, therefore no transparent key. However, the app should not crash and that's a bug.

hhanh00 commented 11 months ago
image

Spendable balance is 0 and it didn't crash.

skironDotNet commented 11 months ago

Sorry the thread title started as T2T because that's what I was originally testing, then wanted to send from sapling as normally you would use the wallet and crashed. The screen you are showing is OK I all understand that now, but past that screen on final confirmation when click send, the wallet just stops, with error mentioned (if started from terminal as ./Ywallet-latest-x86_64.AppImage)

Let me know if there is anything I can do to help investigate that, I can test different distributions, etc. Unfortunately I don't code in any programming language you do so I can't help code/debug. Thanks!

skironDotNet commented 10 months ago

It works now so I don't know what was the problem, maybe the sound thing. Definitely works on new wallet, I think I had some scrambled transaction in the wallet that was crashing the app