Add handling for max uint256 amount in withdrawERC20

[ylv-io]

Description

Type of change

• Bug fix
• New feature
• Breaking change
• Dependency changes
• Deployment
• Forge Script
• Code refactor / cleanup
• Documentation or wording changes
• Other

Checklist:

• The diff is legible and has no extraneous changes
• Complex code has been commented, including external interfaces
• Tests have 100% code coverage
• The base branch is either main, or there's a description of how to merge

Issue Resolution

[openzeppelin-code]

Add handling for max uint256 amount in withdrawERC20

Generated at commit: a095fea86ca0edd8016f60a58f39a0c2a01a56ed

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	4 2 0 12 39 57
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

---

For more details view the full report in OpenZeppelin Code Inspector

[github-actions]

Slither report

THIS CHECKLIST IS NOT COMPLETE. Use --show-ignored-findings to show all the results.
Summary

• unchecked-transfer (1 results) (High)

unchecked-transfer

Impact: High
Confidence: Medium

• ID-0
  AaveWithdrawWorkflow.withdraw(address,uint256) ignores return value by IERC20(asset).transfer(safe,fee)

kinto-core/src/access/workflows/AaveWithdrawWorkflow.sol

Lines 50 to 64 in f0627d1

	function withdraw(address asset, uint256 amount) public returns (uint256) {
	address pool = poolAddressProvider.getPool();
	// If amount is max uint256, withdraw all available
	if (amount == type(uint256).max) {
	amount = IERC20(IAavePool(pool).getReserveData(asset).aTokenAddress).balanceOf(address(this));
	}
	// Withdraw from Aave
	IAavePool(pool).withdraw(asset, amount, address(this));
	// Send the fee to the Safe
	uint256 fee = amount * FEE / 1e18;
	IERC20(asset).transfer(safe, fee);
	return amount - fee;
	}

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.63%. Comparing base (20d6a0d) to head (a095fea).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #333      +/-   ##
==========================================
+ Coverage   88.56%   88.63%   +0.06%     
==========================================
  Files          41       41              
  Lines        2502     2517      +15     
==========================================
+ Hits         2216     2231      +15     
  Misses        286      286              

Files with missing lines	Coverage Δ
src/access/workflows/AaveWithdrawWorkflow.sol	100.00% <100.00%> (ø)
src/access/workflows/WithdrawWorkflow.sol	95.23% <100.00%> (+5.23%)	⬆️

[rrecuero]

to change the fee we will just upgrade?