aboutsummaryrefslogtreecommitdiffstats
path: root/packages/contracts/src
diff options
context:
space:
mode:
authorAmir Bandeali <abandeali1@gmail.com>2018-05-01 07:44:38 +0800
committerGitHub <noreply@github.com>2018-05-01 07:44:38 +0800
commitb5c4b81aacaca83d8629718ff0357ccc3bba4698 (patch)
tree200a715d34d98ae7ead191e20f57d41827ecd411 /packages/contracts/src
parent63ad2ebf0bb9b78bbfc2190390fab83e7c0639b6 (diff)
parentc849c8ef086195c3f24314587772550ee309eb5d (diff)
downloaddexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.tar
dexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.tar.gz
dexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.tar.bz2
dexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.tar.lz
dexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.tar.xz
dexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.tar.zst
dexon-sol-tools-b5c4b81aacaca83d8629718ff0357ccc3bba4698.zip
Merge pull request #563 from 0xProject/feature/contracts/proxyIds
Add hard coded proxyId into each AssetProxy
Diffstat (limited to 'packages/contracts/src')
-rw-r--r--packages/contracts/src/contracts/current/protocol/AssetProxy/IAssetProxy.sol7
-rw-r--r--packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC20Proxy.sol18
-rw-r--r--packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC721Proxy.sol18
-rw-r--r--packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol19
-rw-r--r--packages/contracts/src/contracts/current/protocol/Exchange/mixins/MAssetProxyDispatcher.sol3
5 files changed, 56 insertions, 9 deletions
diff --git a/packages/contracts/src/contracts/current/protocol/AssetProxy/IAssetProxy.sol b/packages/contracts/src/contracts/current/protocol/AssetProxy/IAssetProxy.sol
index 60e74723d..43f45d200 100644
--- a/packages/contracts/src/contracts/current/protocol/AssetProxy/IAssetProxy.sol
+++ b/packages/contracts/src/contracts/current/protocol/AssetProxy/IAssetProxy.sol
@@ -33,4 +33,11 @@ contract IAssetProxy is IAuthorizable {
address to,
uint256 amount)
external;
+
+ /// @dev Gets the proxy id associated with the proxy address.
+ /// @return Proxy id.
+ function getProxyId()
+ external
+ view
+ returns (uint8);
}
diff --git a/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC20Proxy.sol b/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC20Proxy.sol
index eef3a39db..c3cfd8c2e 100644
--- a/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC20Proxy.sol
+++ b/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC20Proxy.sol
@@ -29,6 +29,8 @@ contract ERC20Proxy is
IAssetProxy
{
+ uint8 constant PROXY_ID = 1;
+
/// @dev Transfers ERC20 tokens. Either succeeds or throws.
/// @param assetMetadata ERC20-encoded byte array.
/// @param from Address to transfer token from.
@@ -42,9 +44,25 @@ contract ERC20Proxy is
external
onlyAuthorized
{
+ // Data must be intended for this proxy.
+ require(uint8(assetMetadata[0]) == PROXY_ID);
+
+ // Decode metadata.
require(assetMetadata.length == 21);
address token = readAddress(assetMetadata, 1);
+
+ // Transfer tokens.
bool success = IERC20Token(token).transferFrom(from, to, amount);
require(success == true);
}
+
+ /// @dev Gets the proxy id associated with the proxy address.
+ /// @return Proxy id.
+ function getProxyId()
+ external
+ view
+ returns (uint8)
+ {
+ return PROXY_ID;
+ }
}
diff --git a/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC721Proxy.sol b/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC721Proxy.sol
index eeb7e6710..e11de6744 100644
--- a/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC721Proxy.sol
+++ b/packages/contracts/src/contracts/current/protocol/AssetProxy/proxies/ERC721Proxy.sol
@@ -29,6 +29,8 @@ contract ERC721Proxy is
IAssetProxy
{
+ uint8 constant PROXY_ID = 2;
+
/// @dev Transfers ERC721 tokens. Either succeeds or throws.
/// @param assetMetadata ERC721-encoded byte array
/// @param from Address to transfer token from.
@@ -42,17 +44,31 @@ contract ERC721Proxy is
external
onlyAuthorized
{
+ // Data must be intended for this proxy.
+ require(uint8(assetMetadata[0]) == PROXY_ID);
+
// There exists only 1 of each token.
require(amount == 1);
- // Decode metadata
+ // Decode metadata.
require(assetMetadata.length == 53);
address token = readAddress(assetMetadata, 1);
uint256 tokenId = readUint256(assetMetadata, 21);
+ // Transfer token.
// Either succeeds or throws.
// @TODO: Call safeTransferFrom if there is additional
// data stored in `assetMetadata`.
ERC721Token(token).transferFrom(from, to, tokenId);
}
+
+ /// @dev Gets the proxy id associated with the proxy address.
+ /// @return Proxy id.
+ function getProxyId()
+ external
+ view
+ returns (uint8)
+ {
+ return PROXY_ID;
+ }
}
diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol b/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol
index b03bf464a..b73c6ca90 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol
@@ -43,7 +43,7 @@ contract MixinAssetProxyDispatcher is
{
// Do nothing if no amount should be transferred.
if (amount > 0) {
- // Lookup asset proxy
+ // Lookup asset proxy.
require(assetMetadata.length >= 1);
uint8 assetProxyId = uint8(assetMetadata[0]);
IAssetProxy assetProxy = assetProxies[assetProxyId];
@@ -54,8 +54,7 @@ contract MixinAssetProxyDispatcher is
}
/// @dev Registers an asset proxy to an asset proxy id.
- /// An id can only be assigned to a single proxy at a given time,
- /// however, an asset proxy may be registered to multiple ids.
+ /// An id can only be assigned to a single proxy at a given time.
/// @param assetProxyId Id to register`newAssetProxy` under.
/// @param newAssetProxy Address of new asset proxy to register, or 0x0 to unset assetProxyId.
/// @param oldAssetProxy Existing asset proxy to overwrite, or 0x0 if assetProxyId is currently unused.
@@ -66,11 +65,19 @@ contract MixinAssetProxyDispatcher is
external
onlyOwner
{
- // Ensure the existing asset proxy is not unintentionally overwritten
+ // Ensure the existing asset proxy is not unintentionally overwritten.
require(oldAssetProxy == address(assetProxies[assetProxyId]));
- // Add asset proxy and log registration
- assetProxies[assetProxyId] = IAssetProxy(newAssetProxy);
+ IAssetProxy assetProxy = IAssetProxy(newAssetProxy);
+
+ // Ensure that the id of newAssetProxy matches the passed in assetProxyId, unless it is being reset to 0.
+ if (newAssetProxy != address(0)) {
+ uint8 newAssetProxyId = assetProxy.getProxyId();
+ require(newAssetProxyId == assetProxyId);
+ }
+
+ // Add asset proxy and log registration.
+ assetProxies[assetProxyId] = assetProxy;
emit AssetProxySet(assetProxyId, newAssetProxy, oldAssetProxy);
}
diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/mixins/MAssetProxyDispatcher.sol b/packages/contracts/src/contracts/current/protocol/Exchange/mixins/MAssetProxyDispatcher.sol
index 6d0a3cd38..0cf750d7d 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/mixins/MAssetProxyDispatcher.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/mixins/MAssetProxyDispatcher.sol
@@ -40,8 +40,7 @@ contract MAssetProxyDispatcher {
internal;
/// @dev Registers an asset proxy to an asset proxy id.
- /// An id can only be assigned to a single proxy at a given time,
- /// however, an asset proxy may be registered to multiple ids.
+ /// An id can only be assigned to a single proxy at a given time.
/// @param assetProxyId Id to register`newAssetProxy` under.
/// @param newAssetProxy Address of new asset proxy to register, or 0x0 to unset assetProxyId.
/// @param oldAssetProxy Existing asset proxy to overwrite, or 0x0 if assetProxyId is currently unused.