namada icon indicating copy to clipboard operation
namada copied to clipboard

Tx broadcaster race condition on shutdown

Open sug0 opened this issue 2 years ago • 3 comments

There is a race condition in the tx broadcaster. It's possible to request the ledger to shut down, while the shell is still comitting. Meanwhile, the broadcaster receives a shutdown signal and aborts. If txs are queued in the commit, they won't actually be broadcasted because the broadcaster task has already been killed.

image

sug0 avatar Oct 03 '23 09:10 sug0

@sug0 is this still applicable?

cwgoes avatar Feb 05 '24 09:02 cwgoes

I don't know, but afaik there has been no effort towards fixing this bug

sug0 avatar Feb 05 '24 18:02 sug0

#2598 indirectly patches up this problem, but doesn't necessarily fix it. this is because while the broadcaster will still be executing, it won't be receiving any txs to broadcast. at the time of this writing, only ethereum bridge protocol txs are sent by the broadcaster

sug0 avatar Feb 19 '24 09:02 sug0