Skip to content

Commit 86d477f

Browse files
Merge branch 'fix/FixStartAsPaused' of github.com:dashpay/platform into fix/FixStartAsPaused
2 parents aed065d + 928d5f0 commit 86d477f

File tree

5 files changed

+11
-8
lines changed
  • packages/rs-dpp/src
    • errors/consensus/basic/token
    • state_transition/state_transitions/document/batch_transition/batched_transition
      • token_burn_transition/validate_structure/v0
      • token_direct_purchase_transition/validate_structure/v0
      • token_mint_transition/validate_structure/v0
      • token_transfer_transition/validate_structure/v0

5 files changed

+11
-8
lines changed

packages/rs-dpp/src/errors/consensus/basic/token/invalid_token_amount_error.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ use thiserror::Error;
99
Error, Debug, Clone, PartialEq, Eq, Encode, Decode, PlatformSerialize, PlatformDeserialize,
1010
)]
1111
#[error(
12-
"Invalid token amount {}, exceeds maximum allowed {}",
12+
"Invalid token amount {}, should be between 1 and maximum allowed {}",
1313
token_amount,
1414
max_token_amount
1515
)]

packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_burn_transition/validate_structure/v0/mod.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
use crate::consensus::basic::token::{InvalidTokenAmountError, InvalidTokenNoteTooBigError};
22
use crate::consensus::basic::BasicError;
33
use crate::consensus::ConsensusError;
4+
use crate::data_contract::associated_token::token_perpetual_distribution::distribution_function::MAX_DISTRIBUTION_PARAM;
45
use crate::state_transition::batch_transition::token_burn_transition::v0::v0_methods::TokenBurnTransitionV0Methods;
56
use crate::state_transition::batch_transition::TokenBurnTransition;
67
use crate::tokens::MAX_TOKEN_NOTE_LEN;
@@ -12,10 +13,10 @@ pub(super) trait TokenBurnTransitionActionStructureValidationV0 {
1213
}
1314
impl TokenBurnTransitionActionStructureValidationV0 for TokenBurnTransition {
1415
fn validate_structure_v0(&self) -> Result<SimpleConsensusValidationResult, ProtocolError> {
15-
if self.burn_amount() > i64::MAX as u64 {
16+
if self.burn_amount() > MAX_DISTRIBUTION_PARAM || self.burn_amount() == 0 {
1617
return Ok(SimpleConsensusValidationResult::new_with_error(
1718
ConsensusError::BasicError(BasicError::InvalidTokenAmountError(
18-
InvalidTokenAmountError::new(i64::MAX as u64, self.burn_amount()),
19+
InvalidTokenAmountError::new(MAX_DISTRIBUTION_PARAM, self.burn_amount()),
1920
)),
2021
));
2122
}

packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_direct_purchase_transition/validate_structure/v0/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ pub(super) trait TokenDirectPurchaseTransitionActionStructureValidationV0 {
1212
}
1313
impl TokenDirectPurchaseTransitionActionStructureValidationV0 for TokenDirectPurchaseTransition {
1414
fn validate_structure_v0(&self) -> Result<SimpleConsensusValidationResult, ProtocolError> {
15-
if self.token_count() > MAX_DISTRIBUTION_PARAM {
15+
if self.token_count() > MAX_DISTRIBUTION_PARAM || self.token_count() == 0 {
1616
return Ok(SimpleConsensusValidationResult::new_with_error(
1717
ConsensusError::BasicError(BasicError::InvalidTokenAmountError(
1818
InvalidTokenAmountError::new(MAX_DISTRIBUTION_PARAM, self.token_count()),

packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_mint_transition/validate_structure/v0/mod.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
use crate::consensus::basic::token::{InvalidTokenAmountError, InvalidTokenNoteTooBigError};
22
use crate::consensus::basic::BasicError;
33
use crate::consensus::ConsensusError;
4+
use crate::data_contract::associated_token::token_perpetual_distribution::distribution_function::MAX_DISTRIBUTION_PARAM;
45
use crate::state_transition::batch_transition::token_mint_transition::v0::v0_methods::TokenMintTransitionV0Methods;
56
use crate::state_transition::batch_transition::TokenMintTransition;
67
use crate::tokens::MAX_TOKEN_NOTE_LEN;
@@ -12,10 +13,10 @@ pub(super) trait TokenMintTransitionActionStructureValidationV0 {
1213
}
1314
impl TokenMintTransitionActionStructureValidationV0 for TokenMintTransition {
1415
fn validate_structure_v0(&self) -> Result<SimpleConsensusValidationResult, ProtocolError> {
15-
if self.amount() > i64::MAX as u64 {
16+
if self.amount() > MAX_DISTRIBUTION_PARAM || self.amount() == 0 {
1617
return Ok(SimpleConsensusValidationResult::new_with_error(
1718
ConsensusError::BasicError(BasicError::InvalidTokenAmountError(
18-
InvalidTokenAmountError::new(i64::MAX as u64, self.amount()),
19+
InvalidTokenAmountError::new(MAX_DISTRIBUTION_PARAM, self.amount()),
1920
)),
2021
));
2122
}

packages/rs-dpp/src/state_transition/state_transitions/document/batch_transition/batched_transition/token_transfer_transition/validate_structure/v0/mod.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ use platform_value::Identifier;
22
use crate::consensus::basic::BasicError;
33
use crate::consensus::basic::token::{InvalidTokenAmountError, InvalidTokenNoteTooBigError, TokenTransferToOurselfError};
44
use crate::consensus::ConsensusError;
5+
use crate::data_contract::associated_token::token_perpetual_distribution::distribution_function::MAX_DISTRIBUTION_PARAM;
56
use crate::ProtocolError;
67
use crate::state_transition::batch_transition::token_base_transition::token_base_transition_accessors::TokenBaseTransitionAccessors;
78
use crate::state_transition::batch_transition::token_base_transition::v0::v0_methods::TokenBaseTransitionV0Methods;
@@ -21,10 +22,10 @@ impl TokenTransferTransitionActionStructureValidationV0 for TokenTransferTransit
2122
&self,
2223
owner_id: Identifier,
2324
) -> Result<SimpleConsensusValidationResult, ProtocolError> {
24-
if self.amount() > i64::MAX as u64 {
25+
if self.amount() > MAX_DISTRIBUTION_PARAM || self.amount() == 0 {
2526
return Ok(SimpleConsensusValidationResult::new_with_error(
2627
ConsensusError::BasicError(BasicError::InvalidTokenAmountError(
27-
InvalidTokenAmountError::new(i64::MAX as u64, self.amount()),
28+
InvalidTokenAmountError::new(MAX_DISTRIBUTION_PARAM, self.amount()),
2829
)),
2930
));
3031
}

0 commit comments

Comments
 (0)