Fix setProtocolFactory() (#689)

Author: satyamakgec

contracts/SecurityTokenRegistry.sol

@@ -872,6 +872,8 @@ contract SecurityTokenRegistry is EternalStorage, Proxy {
     function _setProtocolFactory(address _STFactoryAddress, uint8 _major, uint8 _minor, uint8 _patch) internal {
         require(_STFactoryAddress != address(0), "Bad address");
         uint24 _packedVersion = VersionUtils.pack(_major, _minor, _patch);
+        address stFactoryAddress = getAddressValue(Encoder.getKey("protocolVersionST", uint256(_packedVersion)));
+        require(stFactoryAddress == address(0), "Already exists");
         set(Encoder.getKey("protocolVersionST", uint256(_packedVersion)), _STFactoryAddress);
         emit ProtocolFactorySet(_STFactoryAddress, _major, _minor, _patch);
     }

test/n_security_token_registry.js

@@ -683,7 +683,10 @@ contract("SecurityTokenRegistry", async (accounts) => {
             assert.equal(_protocol[0], 2);
             assert.equal(_protocol[1], 2);
             assert.equal(_protocol[2], 0);
-            await I_STRProxied.setProtocolFactory(I_STFactory.address, new BN(3), new BN(0), new BN(0), { from: account_polymath});
+            await catchRevert(
+                I_STRProxied.setProtocolFactory(I_STFactory.address, new BN(3), new BN(0), new BN(0), { from: account_polymath})
+            );
+            await I_STRProxied.setProtocolFactory(I_STFactory.address, new BN(3), new BN(0), new BN(1), { from: account_polymath});
             await I_STRProxied.setLatestVersion(new BN(3), new BN(0), new BN(0), { from: account_polymath});
             _protocol = await I_Getter.getLatestProtocolVersion.call();
             assert.equal(_protocol[0], 3);