Adding slither incorrect-equality disable annotation

dimpar · dimpar · commit 41c8b9d7e248 · 2023-05-24T11:22:21.000+02:00
This require statement breaks slither checks. It is used in other places
in this contract and can be used for this case as well to disable
slither check. The actual solidity code was already merged some time ago
and works as expected.
diff --git a/contracts/staking/TokenStaking.sol b/contracts/staking/TokenStaking.sol
@@ -215,6 +215,7 @@ contract TokenStaking is Initializable, IStaking, Checkpoints {
     }
 
     modifier onlyOwnerOf(address stakingProvider) {
+        // slither-disable-next-line incorrect-equality
         require(
             stakingProviders[stakingProvider].owner == msg.sender,
             "Caller is not owner"

Original file line number	Diff line number	Diff line change
`@@ -215,6 +215,7 @@ contract TokenStaking is Initializable, IStaking, Checkpoints {`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`modifier onlyOwnerOf(address stakingProvider) {`
	`218`	`+ // slither-disable-next-line incorrect-equality`
`218`	`219`	`require(`
`219`	`220`	`stakingProviders[stakingProvider].owner == msg.sender,`
`220`	`221`	`"Caller is not owner"`