search

SiaFoundation / hostd

The next-generation of Sia storage providing software. More performant. More stable. Support for larger nodes.
https://sia.tech/software/hostd
MIT License
39 stars 11 forks source link

Merge chain manager and tpool in SingleAddressWallet #387

Closed n8maninger closed 6 months ago

n8maninger commented 6 months ago

@peterjan, is this enough to fix the race condition, or do I need to change the deferred release to only be called when the transaction should be dropped?

peterjan commented 6 months ago

@n8maninger unfortunately merging the tpool into the manager doesn't fix the race condition. I wasn't sure if that would made you revert the change entirely but I took the liberty to add a commit that release only on error. Feel free to revert that though.