No transfer asset or trx to yourself

Author: zergweak

src/main/java/org/tron/core/actuator/ParticipateAssetIssueActuator.java

@@ -97,7 +97,8 @@ public boolean validate() throws ContractValidateException {
         throw new ContractValidateException("Trx Num must be positive!");
       }
 
-      if ( participateAssetIssueContract.getOwnerAddress().equals(participateAssetIssueContract.getToAddress())){
+      if (participateAssetIssueContract.getOwnerAddress()
+          .equals(participateAssetIssueContract.getToAddress())) {
         throw new ContractValidateException("Cannot participate asset Issue yourself !");
       }
 
@@ -142,7 +143,8 @@ public boolean validate() throws ContractValidateException {
       if (exchangeAmount == 0) {
         throw new ContractValidateException("Can not process the exchange!");
       }
-      AccountCapsule toAccount = this.dbManager.getAccountStore().get(participateAssetIssueContract.getToAddress().toByteArray());
+      AccountCapsule toAccount = this.dbManager.getAccountStore()
+          .get(participateAssetIssueContract.getToAddress().toByteArray());
       if (!toAccount.assetBalanceEnough(assetIssueCapsule.getName(), exchangeAmount)) {
         throw new ContractValidateException("Asset balance is not enough !");
       }

src/main/java/org/tron/core/actuator/TransferActuator.java

@@ -59,31 +59,29 @@ public boolean validate() throws ContractValidateException {
             "contract type error,expected type [TransferContract],real type[" + contract
                 .getClass() + "]");
       }
-
       TransferContract transferContract = this.contract.unpack(TransferContract.class);
-      AccountCapsule ownerAccount = dbManager.getAccountStore()
-          .get(transferContract.getOwnerAddress().toByteArray());
       Preconditions.checkNotNull(transferContract.getOwnerAddress(), "OwnerAddress is null");
       Preconditions.checkNotNull(transferContract.getToAddress(), "ToAddress is null");
       Preconditions.checkNotNull(transferContract.getAmount(), "Amount is null");
 
-      AccountCapsule accountCapsule = dbManager.getAccountStore()
+      if (transferContract.getOwnerAddress().equals(transferContract.getToAddress())) {
+        throw new ContractValidateException("Cannot transfer trx to yourself.");
+      }
+      if (!dbManager.getAccountStore().has(transferContract.getOwnerAddress().toByteArray())) {
+        throw new ContractValidateException("Validate TransferContract error, no OwnerAccount.");
+      }
+
+      AccountCapsule ownerAccount = dbManager.getAccountStore()
           .get(transferContract.getOwnerAddress().toByteArray());
 
-      long balance = accountCapsule.getBalance();
-      long laststOperationTime = accountCapsule.getLatestOperationTime();
+      long balance = ownerAccount.getBalance();
+      long laststOperationTime = ownerAccount.getLatestOperationTime();
       long now = System.currentTimeMillis();
-
+      //TODO:
       //if (now - laststOperationTime < balance) {
       //throw new ContractValidateException();
       //}
 
-      {
-        if (!dbManager.getAccountStore().has(transferContract.getOwnerAddress().toByteArray())) {
-          throw new ContractValidateException("Validate TransferContract error, no OwnerAccount.");
-        }
-      }
-
       // if account with to_address is not existed,  create it.
       ByteString toAddress = transferContract.getToAddress();
       if (!dbManager.getAccountStore().has(toAddress.toByteArray())) {

src/main/java/org/tron/core/actuator/TransferAssetActuator.java

@@ -15,6 +15,7 @@
 
 package org.tron.core.actuator;
 
+import com.google.common.base.Preconditions;
 import com.google.protobuf.Any;
 import com.google.protobuf.ByteString;
 import com.google.protobuf.InvalidProtocolBufferException;
@@ -57,7 +58,9 @@ public boolean execute(TransactionResultCapsule ret) throws ContractExeException
       long amount = transferAssetContract.getAmount();
 
       AccountCapsule ownerAccountCapsule = accountStore.get(ownerKey);
-      ownerAccountCapsule.reduceAssetAmount(assertName, amount);
+      if (!ownerAccountCapsule.reduceAssetAmount(assertName, amount)) {
+        throw new ContractExeException("reduceAssetAmount failed !");
+      }
       accountStore.put(ownerKey, ownerAccountCapsule);
 
       AccountCapsule toAccountCapsule = accountStore.get(toKey);
@@ -78,6 +81,13 @@ public boolean validate() throws ContractValidateException {
       TransferAssetContract transferAssetContract = this.contract
           .unpack(TransferAssetContract.class);
 
+      Preconditions.checkNotNull(transferAssetContract.getOwnerAddress(), "OwnerAddress is null");
+      Preconditions.checkNotNull(transferAssetContract.getToAddress(), "ToAddress is null");
+      Preconditions.checkNotNull(transferAssetContract.getAssetName(), "AssetName is null");
+      Preconditions.checkNotNull(transferAssetContract.getAmount(), "Amount is null");
+      if (transferAssetContract.getOwnerAddress().equals(transferAssetContract.getToAddress())) {
+        throw new ContractValidateException("Cannot transfer asset to yourself.");
+      }
       byte[] ownerKey = transferAssetContract.getOwnerAddress().toByteArray();
       if (!this.dbManager.getAccountStore().has(ownerKey)) {
         throw new ContractValidateException();

src/test/java/org/tron/core/actuator/ParticipateAssetIssueActuatorTest.java

@@ -106,7 +106,7 @@ private Any getContract(long count) {
 
   private void initAssetIssue(long startTimestmp, long endTimestmp) {
     AssetIssueContract assetIssueContract = AssetIssueContract.newBuilder()
-        .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)))
+        .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(TO_ADDRESS)))
         .setName(ByteString.copyFrom(ByteArray.fromString(ASSET_NAME)))
         .setTotalSupply(TOTAL_SUPPLY)
         .setTrxNum(TRX_NUM)