Qpid
  1. Qpid
  2. QPID-4417

swig 1.3.29 generated bad perl bindings

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.18
    • Fix Version/s: 0.19
    • Component/s: None
    • Labels:
      None

      Description

      swig-1.3.29-2.el5.x86_64.rpm (RHEL5) generates bad bindings for getContent, which uses sv_setpv rather than sv_setpvn, and also fiddles with NULL at the end, so fails if the message contains a NULL byte.

      Maybe configure should require a later version?

      1. swig.diff
        0.5 kB
        Jimmy Jones

        Activity

        Hide
        Jimmy Jones added a comment -

        Issue was fixed in SWIG 1.3.32

        Show
        Jimmy Jones added a comment - Issue was fixed in SWIG 1.3.32
        Hide
        Darryl L. Pierce added a comment -

        Okay, so I'll close this ticket as "Won't Fix". On platforms such as RHEL5 and derivatives, please ask them to upgrade to 1.3.32.

        Show
        Darryl L. Pierce added a comment - Okay, so I'll close this ticket as "Won't Fix". On platforms such as RHEL5 and derivatives, please ask them to upgrade to 1.3.32.
        Hide
        Darryl L. Pierce added a comment -

        This issue is fixed by an updated release of Swig (1.3.32)

        Show
        Darryl L. Pierce added a comment - This issue is fixed by an updated release of Swig (1.3.32)
        Hide
        Jimmy Jones added a comment -

        Require SWIG 1.3.32

        Show
        Jimmy Jones added a comment - Require SWIG 1.3.32
        Hide
        Jimmy Jones added a comment -

        Will you accept the attached patch?

        Show
        Jimmy Jones added a comment - Will you accept the attached patch?
        Hide
        Darryl L. Pierce added a comment -

        Certainly! I've added your change, along with a similar check in the CMake toolchain, to require a minimum of 1.3.32 of Swig if language bindings are built.

        Show
        Darryl L. Pierce added a comment - Certainly! I've added your change, along with a similar check in the CMake toolchain, to require a minimum of 1.3.32 of Swig if language bindings are built.
        Hide
        Darryl L. Pierce added a comment -

        Changes to both the autotool and cmake build environments added.

        Show
        Darryl L. Pierce added a comment - Changes to both the autotool and cmake build environments added.

          People

          • Assignee:
            Darryl L. Pierce
            Reporter:
            Jimmy Jones
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development