send Eth from custom account [BUIDLEra]

[seddik11]

Environment

Testnet

Provide a brief description of the functionality you're trying to implement.

I want to implement a public function in a custom account that allows to send Eth from the custom account address to the owner

What is the specific issue or error you're encountering?

No errors. However, when i call that function it is executed successfully but no eth is sent from the abstraction account

Can you share the error messages or logs you're receiving, if any?

here is the [transaction](https://goerli.explorer.zksync.io/tx/0xe922142880d30a7801c82ef58955080a0ac906db2f9cc8d72e0ac4e9dde9d097#overview) on goerli calling that function.

Have you made any recent changes to the contract before encountering this issue?

No it's the first implementation. The abstraction account is deployed successfully but not able to send Eth from it. I want a public function so anyone can call it not only the owner

Are there any external libraries or contracts that your contract interacts with?

No

Can you provide the relevant portions of your contract code where the issue is occurring?

function forwardEth() public { (bool success1, ) = payable(owner1).call{value: 1000000000000000}(""); require(success1, "ETH withdrawal failed for owner1"); }

Have you tried to isolate the problem, and if so, what were the results?

No

What steps have you already taken to try to resolve the issue?

no

Repo Link (Optional)

No response

Additional Details

No response

[seddik11]

Hello,
I tried with ER20 tokens but it's the same problem.

function forward(address tokenAddress) external { uint256 balance = IERC20(tokenAddress).balanceOf(address(this)); IERC20(tokenAddress).transfer(owner1, Math.mulDiv(balance, allocation1, 100)); }

[seddik11]

And this the error log when I try to send Dai from the multiSig address (custom account) using this code

let aaTx = await daiWallet.populateTransaction.transfer(
    owner1.address,
    ethers.utils.parseEther("0.00035")
  );

  const gasPrice = await provider.getGasPrice();

  aaTx = {
    ...aaTx,
    from: multisigAddress,
    gasLimit: ethers.utils.parseEther("0.00000000002"),
    gasPrice: gasPrice,
    chainId: (await provider.getNetwork()).chainId,
    nonce: await provider.getTransactionCount(multisigAddress),
    type: 113,
    customData: {
      gasPerPubdata: utils.DEFAULT_GAS_PER_PUBDATA_LIMIT,
    } as types.Eip712Meta,
    value: ethers.BigNumber.from(0),
  };
  const signedTxHash = EIP712Signer.getSignedDigest(aaTx);

  const multiSignature = ethers.utils.concat([
    ethers.utils.joinSignature(owner1._signingKey().signDigest(signedTxHash)),
    ethers.utils.joinSignature(owner2._signingKey().signDigest(signedTxHash)),
  ]);

  aaTx.customData = {
    ...aaTx.customData,
    customSignature: multiSignature,
  }

  const sentTx = await provider.sendTransaction(utils.serialize(aaTx));
  await sentTx.wait();

error log

reason: 'processing response error',
  code: 'SERVER_ERROR',
  body: '{"jsonrpc":"2.0","error":{"code":3,"message":"failed to validate the transaction. reason: Validation revert: Account validation error: Signature length is incorrect","data":"0x"},"id":119}\n',

requestBody: '{"method":"eth_sendRawTransaction","params":["0x71f9011280840ee6b280840ee6b2808401312d00943e7676937a7e96cfb7616f255b9ad9ff47363d4b80b844a9059cbb000000000000000000000000399c068347275b8abfabacc2412edeec80907a4000000000000000000000000000000000000000000000000000013e52b9abe00082011880808201189469a46af3594bf0582d05547d1145cc4595eb27eb82c350c0b882c109f4d238618456d83faba29a02faa14d44e5c8fcb8cae278045fd3749aeaed25862eeb850437d8d40b096b1e8d49f11555203c8c22013781de53371f2e09cb1bc39d4f7b1349a3445c8d1b81de5ea19fd1faeedd672b2ada2f98798d0179c860276b2bb3e2f49e6311a4a458edfd5239c8e2a4d23a0a620be93cafaa28b2e0c41cc0"],"id":119,"jsonrpc":"2.0"}',
  requestMethod: 'POST',
  url: 'https://testnet.era.zksync.dev',
  transaction: {
    type: 113,
    nonce: 0,
    maxPriorityFeePerGas: BigNumber { _hex: '0x0ee6b280', _isBigNumber: true },
    maxFeePerGas: BigNumber { _hex: '0x0ee6b280', _isBigNumber: true },
    gasLimit: BigNumber { _hex: '0x01312d00', _isBigNumber: true },
    to: '0x3e7676937A7E96CFB7616f255b9AD9FF47363D4b',
    value: BigNumber { _hex: '0x00', _isBigNumber: true },
    data: '0xa9059cbb000000000000000000000000399c068347275b8abfabacc2412edeec80907a4000000000000000000000000000000000000000000000000000013e52b9abe000',
    chainId: BigNumber { _hex: '0x0118', _isBigNumber: true },
    from: '0x69A46Af3594bF0582d05547d1145cC4595eB27eB',
    customData: {
      gasPerPubdata: [BigNumber],
      factoryDeps: [],
      customSignature: '0xc109f4d238618456d83faba29a02faa14d44e5c8fcb8cae278045fd3749aeaed25862eeb850437d8d40b096b1e8d49f11555203c8c22013781de53371f2e09cb1bc39d4f7b1349a3445c8d1b81de5ea19fd1faeedd672b2ada2f98798d0179c860276b2bb3e2f49e6311a4a458edfd5239c8e2a4d23a0a620be93cafaa28b2e0c41c',
      paymasterParams: undefined
    },
    hash: '0x86ebc9f366ab55e254656f6a09109af11395e568e64d84ae460b9cbdbfc1df79',
    confirmations: 0
  },
  transactionHash: '0x86ebc9f366ab55e254656f6a09109af11395e568e64d84ae460b9cbdbfc1df79'
}

[seddik11]

I resolved my problem, in fact you must becareful when using this code to get the deployed account address
If you have made some changes to the constructor you should add those parameters with their types accordingly

const multisigAddress = utils.create2Address(
    factory.address,
    await aaFactory.aaBytecodeHash(),
    salt,
    abiCoder.encode(['address', 'address''], [owner1.address, owner2.address])
  );