New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix vault force withdraw bug with burnt stake pool share #1591

Merged

h4x3rotab merged 2 commits into master from fix-burnt-vault-dust-share

Dec 4, 2024

Contributor

h4x3rotab commented Nov 25, 2024 •

edited

Loading

Skip stake pools with missing/dust NFTs during force withdrawal
Clean up invest_pools list by removing pools with no valid investment
Add test case to verify dust investment handling

This fixes a potential panic when a vault tries to force withdraw from a stake pool where its NFT was already burned due to dust shares.


          fix vault force withdraw bug with burnt stake pool share

32992c4

h4x3rotab requested review from kingsleydon and kvinwang

November 25, 2024 08:39

Contributor

kingsleydon commented Nov 25, 2024

LGTM

kingsleydon approved these changes

View reviewed changes


          update rust version

070f53f

h4x3rotab merged commit 309e465 into master

6 of 8 checks passed

h4x3rotab deleted the fix-burnt-vault-dust-share branch

December 4, 2024 01:23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet