DepositBatch does not null out approvals in the end

[hats-bug-reporter]

Github username: @@deadrosesxyz
Twitter username: @deadrosesxyz
Submission hash (on-chain): 0xc5ed71f74eeb4d930c8236894220feab15a74b51a9fb1aed95376f8c49ef2ad9
Severity: medium

Description:
Description
DepositBatch will be unusable with tokens like USDT

Attack Scenario
Tokens like USDT cannot be set a non-zero approval if they already have a non-zero approval set. When depositing via DepositBatch, the entire balance is approved to the Portfolio contract, although it is not necessary that all of it will be used (due to the ratios of the tokens). If the entire approval is not used, next deposit including USDT will fail.

Attachments

1. Proof of Concept (PoC) File

2. Revised Code File (Optional)

[langnavina97]

DUPLICATE #3

@deadrosesxyz