Summary

Usage of Ownable is prone to error and may make the contract unusable.

Vulnerability Details

The ProxyFactory inherits the Ownable contract. Immediate change of the owner is prone to error, and such a mistake is irrecoverable. More secure design involves a two-step process to change privileged roles.

Impact

An immediate set of the owner to an incorrect address would make the contract unusable.

Tools Used

Manual review

Recommendations

Consider inheriting from Ownable2Step and utilize a two-step process to change the owner.