Skip to content

Commit a34d45e

Browse files
authored
Closes #246: Add warning for _msgSender (#287)
1 parent ac82f3a commit a34d45e

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

contracts/GnosisSafe.sol

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ contract GnosisSafe
2121

2222
using GnosisSafeMath for uint256;
2323

24-
string public constant VERSION = "1.2.0";
24+
string public constant VERSION = "1.3.0";
2525

2626
// keccak256(
2727
// "EIP712Domain(uint256 chainId,address verifyingContract)"

contracts/handler/HandlerContext.sol

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ pragma solidity >=0.7.0 <0.9.0;
77
contract HandlerContext {
88

99
// This function does not rely on a trusted forwarder. Use the returned value only to check information against the calling manager.
10+
/// @notice This is only reliable in combination with a FallbackManager that supports this (e.g. Safe contract >=1.3.0).
11+
/// When using this functionality make sure that the linked _manager (aka msg.sender) supports this.
1012
function _msgSender() internal pure returns (address sender) {
1113
// The assembly code is more direct than the Solidity version using `abi.decode`.
1214
assembly { sender := shr(96, calldataload(sub(calldatasize(), 20))) }

0 commit comments

Comments
 (0)