Skip to content
This repository has been archived by the owner on Oct 4, 2019. It is now read-only.

The cli_wallet does not correctly create transaction. #395

Closed
kotbegemot opened this issue Feb 4, 2018 · 10 comments
Closed

The cli_wallet does not correctly create transaction. #395

kotbegemot opened this issue Feb 4, 2018 · 10 comments
Assignees
Labels
Milestone

Comments

@kotbegemot
Copy link

kotbegemot commented Feb 4, 2018

Steps to reproduce
screenshot-2018-02-02-01-26-05

Actual results
screenshot-2018-02-02-00-57-21

Expected result
cli_wallet returns success on create an account.

@kotbegemot kotbegemot added the bug label Feb 4, 2018
@kotbegemot kotbegemot added this to the 0.16.5 milestone Feb 4, 2018
@kotbegemot kotbegemot added this to To Do in 0.16.5 via automation Feb 4, 2018
@afalaleev afalaleev moved this from To Do to In progress in 0.16.5 Feb 6, 2018
@zxcat
Copy link

zxcat commented Feb 7, 2018

It's not cli_wallet problem, JSON-RPC network_broadcast_api broadcast_transaction fails the same way

@afalaleev
Copy link
Member

@zxcat Yes, you are right the problem is in compatibility with old protocol. Tomorrow, we will create an issue with returning of old protocol for network_broadcast_api.

@zxcat
Copy link

zxcat commented Feb 7, 2018

@afalaleev, yeah, I've already discovered it, here is my hotfix #399

@zxcat
Copy link

zxcat commented Feb 7, 2018

Note: #399 fixes only broadcast_transaction (I checked this method via websockets). But cli_wallet uses broadcast_transaction_synchronous, which still don't work. There is something else broken in addition to input parameters, so cli_wallet transaction expires and golosd crashes.

@zxcat
Copy link

zxcat commented Feb 8, 2018

works for me now for good transactions. still crashes if transaction can't be executed (e.g. trying to transfer more funds than have)

@afalaleev
Copy link
Member

@zxcat can you show an example of transaction which crashes?

@afalaleev
Copy link
Member

@zxcat can you confirm that you are see the same as described in #402.

@afalaleev afalaleev moved this from In progress to Done in 0.16.5 Feb 8, 2018
@zxcat
Copy link

zxcat commented Feb 8, 2018

cli_wallet looks ok now, but golosd either crashes or spams with exception loop and crashes then.
Steps to reproduce:

  1. I run testnet, cyberfounder user have no GBG.
  2. Run cli_wallet -C "unlock 123 && transfer cyberfounder \"null\" \"0.123 GBG\" \"{}\" true"
    Result:
    mutex-fail
560008ms th_a       network_broadcast_api.cpp:162 on_applied_block     ] 13 N5boost16exception_detail10clone_implINS0_19error_info_injectorINS_10lock_errorEEEEE: boost: mutex lock failed in pthread_mutex_lock: Invalid argument
rethrow boost: mutex lock failed in pthread_mutex_lock: Invalid argument:
    {"what":"boost: mutex lock failed in pthread_mutex_lock: Invalid argument"}
    th_a  network_broadcast_api.cpp:162 on_applied_block
1560008ms th_a       database.cpp:1012             notify_applied_block ] Caught exception in plugin: 13 N5boost16exception_detail10clone_implINS0_19error_info_injectorINS_10lock_errorEEEEE: boost: mutex lock failed in pthread_mutex_lock: Invalid argument
rethrow boost: mutex lock failed in pthread_mutex_lock: Invalid argument:
    {"what":"boost: mutex lock failed in pthread_mutex_lock: Invalid argument"}
    th_a  network_broadcast_api.cpp:162 on_applied_block

You can see transactions send by running that command, then some blocks produced, then assertion fail in a promise.

Sometimes it crashes instantly, sometimes starts to spam that error message each block.

  1. If it's not crashed yet, executing valid transaction will kill golosd:
    [1] 44278 segmentation fault ../programs/golosd/golosd -d testnet165

@afalaleev
Copy link
Member

@zxcat Thank you 👍In your case golosd crashes in the same place as in case from #402

@zxcat
Copy link

zxcat commented Feb 8, 2018

@afalaleev, yeah, looks the same

@kotbegemot kotbegemot added this to Done in development Mar 30, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
development
  
Done
0.16.5
  
Done
Development

No branches or pull requests

3 participants