Uploaded image for project: 'Qpid'
  1. Qpid
  2. QPID-4417

swig 1.3.29 generated bad perl bindings

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: 0.18
        • Fix Version/s: 0.19
        • Component/s: Perl Client
        • 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. File
            swig.diff
            0.5 kB
            Jimmy Jones

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jimmyjones2 Jimmy Jones added a comment -

            Issue was fixed in SWIG 1.3.32

            Show
            jimmyjones2 Jimmy Jones added a comment - Issue was fixed in SWIG 1.3.32
            Hide
            Permalink
            mcpierce 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
            mcpierce 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
            Permalink
            mcpierce Darryl L. Pierce added a comment -

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

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

            Require SWIG 1.3.32

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

            Will you accept the attached patch?

            Show
            jimmyjones2 Jimmy Jones added a comment - Will you accept the attached patch?
            Hide
            Permalink
            mcpierce 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
            mcpierce 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
            Permalink
            mcpierce Darryl L. Pierce added a comment -

            Changes to both the autotool and cmake build environments added.

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development