negative balance when syncing wallet

[nicbus]

Using commit ac43eb2 plus a cargo update to include the latest fixes, I sometimes run into an issue.

This is what I'm doing:

• issue a NIA asset on wallet 1
• send some NIA assets from wallet 1 to wallet 2
• send some more NIA assets from wallet 1 to wallet 2
• send some NIA assets from wallet 2 to wallet 1

All operations are carried out using opret1st closing method, all invoices are blinded.

During the last transfer, wallet 2 fails to sync with:

Loading descriptor from wallet wallet_2 ... success
Syncing keychain 0 .......... keychain 1 .......... keychain 9 .............thread 'main' panicked at ~/.cargo/git/checkouts/bp-wallet-997f3c057e1b59f2/e05f6c7/src/data.rs:410:55:
negative balance: TryFromIntError(())

Here is the whole data dir for wallet 2, including consignments for previous transfers:
wallet2_data.tar.gz

[dr-orlovsky]

This is entire mystery for me why this happens. I will wait on the confirmation that the issue is still in the released beta 6 and will pay more attention in fixing it. I believe it is a pure command-line tool issue unrelated to a more deep levels

[dr-orlovsky]

Can we verify that the issue is still there with the most recent master branches?

[ghost]

Can we verify that the issue is still there with the most recent master branches?

i have testet it, it fixed by BP-WG/bp-wallet#35

[dr-orlovsky]

@nicbus can you pls confirm that the bug is fixed

[nicbus]

I have tried to reproduce the issue with the latest code and I haven't been able to.

I am not 100% confident that this has been fixed, since the issue was not presenting itself every time even when I reported it, but I think we can close this. I'll open a new issue in case I stumble upon it again.