https://github.com/code-423n4/2024-07-reserve/blob/3f133997e186465f4904553b0f8e86ecb7bbacbf/contracts/p1/Broker.sol#L199
https://github.com/code-423n4/2024-07-reserve/blob/3f133997e186465f4904553b0f8e86ecb7bbacbf/contracts/p1/Broker.sol#L221
When Broker is initialized, and later re-configured via governance calls, it validates the batchAuctionLength and dutchAuctionLength given parameters individually:
It does however not validate the situation when they are both provided as 0. This is a situation that is not admissible because in this case, no auction can be created and a BasketManager cant open recollateralization trades.
Consider adding additional checks in setBatchAuctionLength and setDutchAuctionLength to prevent setting both to 0.
