FourthState / plasma-mvp-sidechain

Cosmos SDK (Tendermint consensus) side-chain implementation for plasma
Apache License 2.0
112 stars 35 forks source link

GetUnspentFromWallet panic's on unspent deposits #190

Open hamdiallam opened 4 years ago

hamdiallam commented 4 years ago

Summary of bug

`(DataStore).GetUnspentFromWallet assumes that all unspent outputs are transaction outputs. When there's an unspent deposit, this causes plasmad to panic.

Expected behavior

GetUnspentFromWallet should return general output information. If wanted, the TxOuput or DepositOutput should be an additional call.

Steps to reproduce

Deposit into the smart contract, create and include-deposit transaction, then call plasmacli query info on the account that owns the deposit.

Versions/External Software

0.2.0-91-g8f898d9