crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix StaticCallProxy comment and add sanity checks to TestStaticCallTarget

Browse Source
This commit is contained in:
Amir Bandeali 2019-06-17 12:22:06 -07:00
parent 5dcf7919fc
commit 45aacf122f
2 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
contracts/asset-proxy/contracts/src/StaticCallProxy.sol
View File

@ -174,7 +174,7 @@ contract StaticCallProxy {
// + 32 (assetData len)
// + 4 (proxyId)
// + 32 (callTarget)
// + 32 (assetDataOffset)
// + 32 (staticCallDataOffset)
let expectedResultHash := calldataload(add(assetDataOffset, 100))
if sub(callResultHash, expectedResultHash) {

13
contracts/asset-proxy/contracts/test/TestStaticCallTarget.sol
View File

@ -18,9 +18,13 @@
pragma solidity ^0.5.5;
import "@0x/contracts-utils/contracts/src/LibBytes.sol";
contract TestStaticCallTarget {
using LibBytes for bytes;
uint256 internal _state;
function updateState()
@ -51,12 +55,17 @@ contract TestStaticCallTarget {
function noInputFunction()
external
pure
{}
{
assert(msg.data.length == 4 && msg.data.readBytes4(0) == bytes4(keccak256("noInputFunction()")));
}
function dynamicInputFunction(bytes calldata a)
external
pure
{}
{
bytes memory abiEncodedData = abi.encodeWithSignature("dynamicInputFunction(bytes)", a);
assert(msg.data.equals(abiEncodedData));
}
function returnComplexType(uint256 a, uint256 b)
external