Conduit can be rendered useless

[code423n4]

Lines of code

https://github.com/code-423n4/2022-05-opensea-seaport/blob/main/contracts/conduit/ConduitController.sol#L57

Vulnerability details

Impact

Currently their is no check to see whether initialOwner is set to address(0) while creating new Conduit. The new Conduit would get created but user wont be able to use it.

Proof of Concept

1. User call createConduit function but accidentally passes address(0) as initialOwner
2. The conduit gets created and owner is set to address(0)

Recommended Mitigation Steps

Add below check in createConduit function:

require(initialOwner!=address(0);, "Incorrect owner");

[0age]

Confirmed that there should be a check (indeed have already implemented this change) but should be low-severity as an improperly configured conduit will simply be unusable and can be redeployed

[HardlyDifficult]

Grouping with the warden's QA report #53