Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.9.2
    • Fix Version/s: 0.9.3
    • Labels:
      None

      Activity

      Hide
      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
      roger.meier Roger Meier added a comment -

      looks good! please commit

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

      +1, thanks Ben Craig

      Show
      jfarrell Jake Farrell added a comment - +1, thanks Ben Craig
      Hide
      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
      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