KomodoPlatform / komodo-wallet

Komodo Wallet
https://app.komodoplatform.com
239 stars 206 forks source link

[BUG]: mm2 crash #2317

Closed ciumete closed 1 year ago

ciumete commented 1 year ago

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Run Atomicdex Desktop ver 0.5.8'
  2. After few seconds mm2.exe crush

Operating Environment(s):

Log file :

29 17:26:27, atomicdex_gossipsub::behaviour:189] INFO Subscribed to topic: orbk/LBC:THC 29 17:26:27, coins::utxo::utxo_builder::utxo_coin_builder:820] INFO Use protocol version 1.4 for Electrum "electrum25.groestlcoin.org:50001" 29 17:26:27, common:500] panicked at 'index out of bounds: the len is 2 but the index is 2', mm2src\coins\utxo\utxo_common.rs:3119:29 29 17:26:27, common:501] backtrace ??:0] BaseThreadInitThunk 0x7ffbba647613 ??:0] RtlUserThreadStart 0x7ffbba7826f0

image

tonymorony commented 1 year ago

thank you for issue opening!

@shamardy could please someone from API team investigate why this crash might happen / can it be actual? 29 17:26:27, common:500] panicked at 'index out of bounds: the len is 2 but the index is 2', mm2src\coins\utxo\utxo_common.rs:3119:29

ciumete commented 1 year ago

thank you for issue opening!

@shamardy could please someone from API team investigate why this crash might happen / can it be actual? 29 17:26:27, common:500] panicked at 'index out of bounds: the len is 2 but the index is 2', mm2src\coins\utxo\utxo_common.rs:3119:29

is still actual, same error

shamardy commented 1 year ago

could please someone from API team investigate why this crash might happen / can it be actual?

This is related to tx history, this can happen if there is a problem with transaction deserialization. It would be good to know which coin and transaction produced this error but this info are not in the logs. I will transfer this issue to the komodefi repo.

shamardy commented 1 year ago

@tonymorony Opened an issue in the related repo https://github.com/KomodoPlatform/komodo-defi-framework/issues/1906

shamardy commented 1 year ago

The PR that fixes this will be in the next API release, once this release is in desktop we can know more about the coin/transactions from the wallet history that caused this problem through the logs. Mm2 will not crash while starting then but some transactions will not be shown in history due to the problem we are trying to pinpoint.

smk762 commented 1 year ago

Test builds for this bug at https://github.com/KomodoPlatform/komodo-wallet-desktop/actions/runs/5876114420

smk762 commented 1 year ago

Closing - Fixed in API which will be in next version of app.