diba-io / bitmask-core

Core functionality for the BitMask wallet
https://bitmask.app
Other
95 stars 21 forks source link

Fix Storage State in Carbonado #351

Closed crisdut closed 1 year ago

crisdut commented 1 year ago

Description

I found a problem with carbonado overwriting file verification when we use wasm32 (see here).

After running consecutive transfers (more than 100x), the rgb_wallet or stock ends up having a minimum variation of bytes downwards, which ends up being unsaved, and this causes some issues in contract states (@josediegorobles I believe this causes some inconsistencies in the balances you reported).

This PR still needs to be completed, I need some help to test it.

crisdut commented 1 year ago

Sure, this should be fine. We really should get an automerge implementation of #268, however, I think that would resolve all these issues.

Yes, we finished that: #353