bartubozkurt · February 1, 2023 16:09
diff --git a/IncorrectAccessControl.sol b/IncorrectAccessControl.sol
 /* Bad */
 address owner;
 function setOwner() public {
 	owner = msg.sender;
 }

 /* Better */
 contract Buggy{
 /* 
 modifier onlyOwner() {
   require(_owner == _msgSender(), "Ownable: caller is not the owner");
   _;
 }
 */
    address owner;

    function set_protected() public onlyOwner(){
        owner = msg.sender;
    }
 }
	/* Bad */
	address owner;
	function setOwner() public {
	owner = msg.sender;
	}

	/* Better */
	contract Buggy{
	/*
	modifier onlyOwner() {
	require(_owner == _msgSender(), "Ownable: caller is not the owner");
	_;
	}
	*/
	address owner;

	function set_protected() public onlyOwner(){
	owner = msg.sender;
	}
	}