Merge pull request #521 from graphprotocol/audit/gns-ownership-n05

Audit: [N05] Unnecessary use of msg.sender
graphprotocol · Dec 1, 2021 · 78b1e39 · 78b1e39
2 parents 237a26d + 66d6e5e
commit 78b1e39
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/contracts/discovery/GNS.sol b/contracts/discovery/GNS.sol
@@ -376,7 +376,7 @@ contract GNS is GNSV2Storage, GraphUpgradeable, IGNS, Multicall {
 
         // Pull tokens from sender
         address curator = msg.sender;
-        TokenUtils.pullTokens(graphToken(), msg.sender, _tokensIn);
+        TokenUtils.pullTokens(graphToken(), curator, _tokensIn);
 
         // Get name signal to mint for tokens deposited
         (uint256 vSignal, ) = curation().mint(subgraphData.subgraphDeploymentID, _tokensIn, 0);