# Sommiler Re-audit

QSP-1: Fixed

QSP-2: Unresolved (Example: https://github.com/VolumeFi/cellars/blob/09d2e7b73eba49457edee7c71f83240e41827f99/contracts/CellarPoolShare.sol#L147)

QSP-3: Fixed
QSP-4: Fixed (in a weird way. Why not just revert?)

QSP-5: Partially Fixed, still returns true irrespective of the outcome of the transfer
QSP-6: Fixed

QSP-7: Fixed

QSP-8: I believe this is unresolved. What are your thoughts Hisham?

QSP-9: Fixed

QSP-10: Acknowledged
QSP-11: Unresolved. This response is not true. `_owner` is set in the constructor at deploy time and can be an EOA address.

QSP-12: Acknowledged

QSP-13: Unresolved. `interfaces.sol` still has clone and own code.

QSP-14: Unresolved

QSP-15: Unresolved

QSP-16: Acknowledged

QSP-17: Fixed


DOC-1: Unresolved

BP-1: Unresolved

BP-2: Unresolved
BP-3: Unresolved

BP-4: Unresolved
BP-5: Fixed

BP-6: Unresolved

BP-7: Present already?

BP-8: Unresolved
BP-9: Unresolved