fix: deposit saturation

r0gue-io · Jan 10, 2025 · 5d350b1 · 5d350b1
1 parent 1cdad44
commit 5d350b1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pallets/nfts/src/features/approvals.rs b/pallets/nfts/src/features/approvals.rs
@@ -237,7 +237,7 @@ impl<T: Config<I>, I: 'static> Pallet<T, I> {
 				let deposit_required = T::CollectionApprovalDeposit::get();
 				let current_deposit =
 					maybe_approval.map(|(_, deposit)| deposit).unwrap_or_default();
-				T::Currency::reserve(&owner, deposit_required - current_deposit)?;
+				T::Currency::reserve(&owner, deposit_required.saturating_sub(current_deposit))?;
 				*maybe_approval = Some((deadline, deposit_required));
 				Ok(())
 			},