omgnetwork / plasma-mvp

OmiseGO's research implementation of Minimal Viable Plasma
MIT License
561 stars 158 forks source link

omg sendtx error #213

Closed victoryeo closed 5 years ago

victoryeo commented 5 years ago

Issue Type

[x] bug report
[ ] feature request

Current Behavior

Call: omg sendtx 1000 0 0 0 0 0 0x0 0x4B3eC6c9dC67079E82152d6D55d8dd96a8e6AA26 2 0x0 0 3bb369fecdc16b93b99514d8ed9c2e87c5824cf4a6a98d2e8e91b7dd0c063304

In chain.py, validate_transaction(self, tx, temp_spent={}): , i print out the blknum, self.blocks, self.blocks[blknum] , transaction_set and txindex. This condition is fulfilled:

        if txindex not in self.blocks[blknum].transaction_set:
            print("txindex not in dictionary")
            return

Expected Behavior

No error. txindex should be in transaction_set

Steps to Reproduce

  1. omg deposit 100 0xfd02EcEE62797e75D86BCff1642EB0844afB28c7
  2. omg sendtx 1000 0 0 0 0 0 0x0 0x4B3eC6c9dC67079E82152d6D55d8dd96a8e6AA26 2 0x0 0 3bb369fecdc16b93b99514d8ed9c2e87c5824cf4a6a98d2e8e91b7dd0c063304

Suggested Fix

Motivation for Change

omg sendtx is not working. It needs to be fixed.

System Specs