forward revert message from low level solidity call












1















I'm working on a proxy contract that will perform calls as follows:



bool success;
bytes memory returndata;
(success, returndata) = _to.call.value(_value)(_data);
require(success);


this works great, except that, if the call reverts with an error message, this function will simply reverts, hiding the reason string.



How can I retrieve the reason string (I assume in the returndata) and use if to revert properly










share|improve this question



























    1















    I'm working on a proxy contract that will perform calls as follows:



    bool success;
    bytes memory returndata;
    (success, returndata) = _to.call.value(_value)(_data);
    require(success);


    this works great, except that, if the call reverts with an error message, this function will simply reverts, hiding the reason string.



    How can I retrieve the reason string (I assume in the returndata) and use if to revert properly










    share|improve this question

























      1












      1








      1


      2






      I'm working on a proxy contract that will perform calls as follows:



      bool success;
      bytes memory returndata;
      (success, returndata) = _to.call.value(_value)(_data);
      require(success);


      this works great, except that, if the call reverts with an error message, this function will simply reverts, hiding the reason string.



      How can I retrieve the reason string (I assume in the returndata) and use if to revert properly










      share|improve this question














      I'm working on a proxy contract that will perform calls as follows:



      bool success;
      bytes memory returndata;
      (success, returndata) = _to.call.value(_value)(_data);
      require(success);


      this works great, except that, if the call reverts with an error message, this function will simply reverts, hiding the reason string.



      How can I retrieve the reason string (I assume in the returndata) and use if to revert properly







      solidity revert-opcode proxy-contracts






      share|improve this question













      share|improve this question











      share|improve this question




      share|improve this question










      asked Apr 1 at 14:02









      AmxxAmxx

      1185




      1185






















          1 Answer
          1






          active

          oldest

          votes


















          3














          It is simpler than you think:



          require(success, string (returndata));


          See documentation for details. You may also forward raw bytes, but you will need to use assembly.






          share|improve this answer
























          • I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

            – Amxx
            Apr 1 at 14:52












          Your Answer








          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "642"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });














          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fethereum.stackexchange.com%2fquestions%2f69133%2fforward-revert-message-from-low-level-solidity-call%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          1 Answer
          1






          active

          oldest

          votes








          1 Answer
          1






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes









          3














          It is simpler than you think:



          require(success, string (returndata));


          See documentation for details. You may also forward raw bytes, but you will need to use assembly.






          share|improve this answer
























          • I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

            – Amxx
            Apr 1 at 14:52
















          3














          It is simpler than you think:



          require(success, string (returndata));


          See documentation for details. You may also forward raw bytes, but you will need to use assembly.






          share|improve this answer
























          • I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

            – Amxx
            Apr 1 at 14:52














          3












          3








          3







          It is simpler than you think:



          require(success, string (returndata));


          See documentation for details. You may also forward raw bytes, but you will need to use assembly.






          share|improve this answer













          It is simpler than you think:



          require(success, string (returndata));


          See documentation for details. You may also forward raw bytes, but you will need to use assembly.







          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Apr 1 at 14:31









          Mikhail VladimirovMikhail Vladimirov

          1,351316




          1,351316













          • I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

            – Amxx
            Apr 1 at 14:52



















          • I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

            – Amxx
            Apr 1 at 14:52

















          I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

          – Amxx
          Apr 1 at 14:52





          I tryed require(success, returndata); and started looking into assembly when it failled to compile ... thanks for proving how stupid I am :P

          – Amxx
          Apr 1 at 14:52


















          draft saved

          draft discarded




















































          Thanks for contributing an answer to Ethereum Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fethereum.stackexchange.com%2fquestions%2f69133%2fforward-revert-message-from-low-level-solidity-call%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          Færeyskur hestur Heimild | Tengill | Tilvísanir | LeiðsagnarvalRossið - síða um færeyska hrossið á færeyskuGott ár hjá færeyska hestinum

          He _____ here since 1970 . Answer needed [closed]What does “since he was so high” mean?Meaning of “catch birds for”?How do I ensure “since” takes the meaning I want?“Who cares here” meaningWhat does “right round toward” mean?the time tense (had now been detected)What does the phrase “ring around the roses” mean here?Correct usage of “visited upon”Meaning of “foiled rail sabotage bid”It was the third time I had gone to Rome or It is the third time I had been to Rome

          Bunad