Missing nonReentrant modifiers

[code423n4]

Handle

pauliax

Vulnerability details

Impact

I expect to see nonReentrant on functions with external calls (e.g. moving tokens). Here is a list (it may not be complete) of functions that do not have this modifier:
Contract SushiYieldSource functions supplyTokenTo, redeemToken. Contract IdleYieldSource function sponsor. Contract BadgerYieldSource functions supplyTokenTo, redeemToken. Contract ATokenYieldSource function sponsor.

Recommended Mitigation Steps

Add nonReentrant modifier to the functions that interact with external actors.

[asselstine]

See #119

[dmvt]

dropping this to a level 1 since no threat is actually defined and the other duplicate issues all agree this is a low risk

[dmvt]

duplicate of #119