Make dispatchTransferFrom revert with reason from AssetProxy on failure

1 files changed, 4 insertions, 5 deletions

diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol b/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol
index 93e752af3..f7086f543 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/MixinAssetProxyDispatcher.sol
@@ -180,13 +180,12 @@ contract MixinAssetProxyDispatcher is
                     cdStart,                // pointer to start of input
                     sub(cdEnd, cdStart),    // length of input  
                     cdStart,                // write output over input
-                    0                       // output size is 0 bytes
+                    512                     // reserve 512 bytes for output
                 )
+                if eq(success, 0) {
+                    revert(cdStart, returndatasize())
+                }
             }
-            require(
-                success,
-                "TRANSFER_FAILED"
-            );
         }
     }
 }