Add revert if the bytecode length is not greater than zero

Author: corydickson

contracts/utils/Create2.sol

@@ -19,6 +19,7 @@ library Create2 {
      */
     function deploy(bytes32 salt, bytes memory bytecode) internal returns (address) {
         address addr;
+        require(bytecode.length > 0, "Create2: bytecode must have a length");
         // solhint-disable-next-line no-inline-assembly
         assembly {
             addr := create2(0, add(bytecode, 0x20), mload(bytecode), salt)

test/utils/Create2.test.js

@@ -59,6 +59,11 @@ describe('Create2', function () {
       this.factory.deploy(saltHex, constructorByteCode, { from: deployerAccount }), 'Create2: Failed on deploy'
     );
   });
+  it('should failed deploying a contract in an existent address', async function () {
+    await expectRevert(
+      this.factory.deploy(saltHex, '0x0', { from: deployerAccount }), 'Create2: bytecode must have a length'
+    );
+  });
 });
 
 function computeCreate2Address (saltHex, bytecode, deployer) {