(migration): require migration amount to be greater than zero

aboutcircles · Oct 1, 2024 · 8c857ce · 8c857ce
1 parent 74a6435
commit 8c857ce
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/errors/Errors.sol b/src/errors/Errors.sol
@@ -96,3 +96,7 @@ interface INameRegistryErrors {
 
     error CirclesNamesOrganizationHasNoSymbol(address organization, uint8 code);
 }
+
+interface IMigrationErrors {
+    error CirclesMigrationAmountMustBeGreaterThanZero();
+}
diff --git a/src/migration/Migration.sol b/src/migration/Migration.sol
@@ -6,7 +6,7 @@ import "../hub/IHub.sol";
 import "./IHub.sol";
 import "./IToken.sol";
 
-contract Migration is ICirclesErrors {
+contract Migration is ICirclesErrors, IMigrationErrors {
     // Constant
 
     uint256 private constant ACCURACY = uint256(10 ** 8);
@@ -75,6 +75,10 @@ contract Migration is ICirclesErrors {
                 // Invalid avatar, not registered in hub V1.
                 revert CirclesAddressCannotBeZero(2);
             }
+            if (_amounts[i] == 0) {
+                // Amount must be greater than zero.
+                revert CirclesMigrationAmountMustBeGreaterThanZero();
+            }
             convertedAmounts[i] = convertFromV1ToDemurrage(_amounts[i]);
             // transfer the v1 Circles to this contract to be locked
             circlesV1.transferFrom(msg.sender, address(this), _amounts[i]);