From d77da248e7e6851a4e9a29d548910f978added0a Mon Sep 17 00:00:00 2001
From: Vito <5780819+Tapanito@users.noreply.github.com>
Date: Wed, 24 Jun 2026 12:13:27 +0200
Subject: [PATCH] address review feedback

---
 .../tx/transactors/dex/AMMWithdraw.cpp        | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/src/libxrpl/tx/transactors/dex/AMMWithdraw.cpp b/src/libxrpl/tx/transactors/dex/AMMWithdraw.cpp
index 22c95f51a3..32cde78e91 100644
--- a/src/libxrpl/tx/transactors/dex/AMMWithdraw.cpp
+++ b/src/libxrpl/tx/transactors/dex/AMMWithdraw.cpp
@@ -323,15 +323,16 @@ AMMWithdraw::issuerFreezeHandling() const
     // When the withdrawer is the issuer of a pool asset, the issuer can
     // always receive their own token — even when the pool is frozen.
     // Use IgnoreFreeze so ammHolds returns real balances instead of zero.
-    if (ctx_.view().rules().enabled(fixCleanup3_3_0))
-    {
-        auto const asset1 = Asset{ctx_.tx[sfAsset]};
-        auto const asset2 = Asset{ctx_.tx[sfAsset2]};
-        if (!asset1.native() && accountID_ == asset1.getIssuer())
-            return FreezeHandling::IgnoreFreeze;
-        if (!asset2.native() && accountID_ == asset2.getIssuer())
-            return FreezeHandling::IgnoreFreeze;
-    }
+    if (!ctx_.view().rules().enabled(fixCleanup3_3_0))
+        return FreezeHandling::ZeroIfFrozen;
+
+    auto const asset1 = Asset{ctx_.tx[sfAsset]};
+    auto const asset2 = Asset{ctx_.tx[sfAsset2]};
+    if (!asset1.native() && accountID_ == asset1.getIssuer())
+        return FreezeHandling::IgnoreFreeze;
+    if (!asset2.native() && accountID_ == asset2.getIssuer())
+        return FreezeHandling::IgnoreFreeze;
+
     return FreezeHandling::ZeroIfFrozen;
 }