diff --git a/contracts/tokens/SecurityToken.sol b/contracts/tokens/SecurityToken.sol index 99aa375e9..59e529575 100644 --- a/contracts/tokens/SecurityToken.sol +++ b/contracts/tokens/SecurityToken.sol @@ -577,10 +577,6 @@ contract SecurityToken is ERC20, ERC20Detailed, Ownable, ReentrancyGuard, Securi */ function setController(address _controller) public onlyOwner { require(_isControllable()); - // Below condition is to restrict the owner/issuer to become the controller(In an ideal world). - // But for non ideal case issuer could set another address which is not the owner of the token - // but issuer holds its private key. - require(_controller != msg.sender); emit SetController(controller, _controller); controller = _controller; } diff --git a/test/c_checkpoints.js b/test/c_checkpoints.js index 168e1e8e3..2526539f6 100644 --- a/test/c_checkpoints.js +++ b/test/c_checkpoints.js @@ -140,13 +140,7 @@ contract("Checkpoints", async function(accounts) { assert.equal(web3.utils.toAscii(log.args._name).replace(/\u0000/g, ""), "GeneralTransferManager"); }); - it("Should set controller to token owner --failed not allowed", async () => { - await catchRevert( - I_SecurityToken.setController(token_owner, { from: token_owner }) - ); - }); - - it("Should set the tcontroller", async() => { + it("Should set the controller", async() => { await I_SecurityToken.setController(account_controller, {from: token_owner}); })