Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-3231

CPP: Limit recursion depth to 64

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 0.9.2
        • Fix Version/s: 0.9.3
        • Component/s: C++ - Compiler, C++ - Library
        • Labels:
          None

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          githubbot ASF GitHub Bot added a comment -

          GitHub user ben-craig opened a pull request:

          https://github.com/apache/thrift/pull/546

          THRIFT-3231 CPP: Limit recursion depth to 64

          Client: cpp
          Patch: Ben Craig <bencraig@apache.org>

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/ben-craig/thrift THRIFT-3231

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/thrift/pull/546.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #546

          commit cfaadcc4adcfde2a8232c62ec89870b73ef40df1
          Author: Ben Craig <bencraig@apache.org>
          Date: 2015-07-09T01:50:33Z

          THRIFT-3231 CPP: Limit recursion depth to 64
          Client: cpp
          Patch: Ben Craig <bencraig@apache.org>

          Show
          githubbot ASF GitHub Bot added a comment - GitHub user ben-craig opened a pull request: https://github.com/apache/thrift/pull/546 THRIFT-3231 CPP: Limit recursion depth to 64 Client: cpp Patch: Ben Craig <bencraig@apache.org> You can merge this pull request into a Git repository by running: $ git pull https://github.com/ben-craig/thrift THRIFT-3231 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/thrift/pull/546.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #546 commit cfaadcc4adcfde2a8232c62ec89870b73ef40df1 Author: Ben Craig <bencraig@apache.org> Date: 2015-07-09T01:50:33Z THRIFT-3231 CPP: Limit recursion depth to 64 Client: cpp Patch: Ben Craig <bencraig@apache.org>
          Hide
          Permalink
          roger.meier Roger Meier added a comment -

          looks good! please commit

          Show
          roger.meier Roger Meier added a comment - looks good! please commit
          Hide
          Permalink
          jfarrell Jake Farrell added a comment -

          +1, thanks Ben Craig

          Show
          jfarrell Jake Farrell added a comment - +1, thanks Ben Craig
          Hide
          Permalink
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/thrift/pull/546

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/thrift/pull/546
          Hide
          Permalink
          hudson Hudson added a comment -

          SUCCESS: Integrated in Thrift #1599 (See https://builds.apache.org/job/Thrift/1599/)
          THRIFT-3231 CPP: Limit recursion depth to 64 (bencraig: rev cfaadcc4adcfde2a8232c62ec89870b73ef40df1)

          • compiler/cpp/src/generate/t_cpp_generator.cc
          • lib/cpp/CMakeLists.txt
          • lib/cpp/src/thrift/protocol/TProtocol.h
          • lib/cpp/Makefile.am
          • lib/cpp/src/thrift/protocol/TProtocol.cpp
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Thrift #1599 (See https://builds.apache.org/job/Thrift/1599/ ) THRIFT-3231 CPP: Limit recursion depth to 64 (bencraig: rev cfaadcc4adcfde2a8232c62ec89870b73ef40df1) compiler/cpp/src/generate/t_cpp_generator.cc lib/cpp/CMakeLists.txt lib/cpp/src/thrift/protocol/TProtocol.h lib/cpp/Makefile.am lib/cpp/src/thrift/protocol/TProtocol.cpp

            People

            • Assignee:
              ben.craig Ben Craig
              Reporter:
              ben.craig Ben Craig
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development