Qpid Proton
  1. Qpid Proton
  2. PROTON-202

build on non-x86 fails with "error: 'scanned' may be used uninitialized"

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.3
    • Fix Version/s: None
    • Component/s: proton-c
    • Labels:
      None

      Description

      Build of qpid-proton-0.3 fails on s390 and ppc(64) with an error from gcc

      ...
      ./usr/bin/cc -Dqpid_proton_EXPORTS -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m31 -march=z9-109 -mtune=z10 -fPIC -I/builddir/build/BUILD/qpid-proton-c-0.3 -I/builddir/build/BUILD/qpid-proton-c-0.3/include -Werror -Wall -pedantic-errors -std=c99 -o CMakeFiles/qpid-proton.dir/src/messenger.c.o -c /builddir/build/BUILD/qpid-proton-c-0.3/src/messenger.c
      ............/builddir/build/BUILD/qpid-proton-c-0.3/src/codec/codec.c: In function 'pn_data_vscan':
      /builddir/build/BUILD/qpid-proton-c-0.3/src/codec/codec.c:1712:16: error: 'scanned' may be used uninitialized in this function [-Werror=maybe-uninitialized]
      cc1: all warnings being treated as errors
      make[2]: *** [CMakeFiles/qpid-proton.dir/src/codec/codec.c.o] Error 1
      ...

      Probably the understanding of the function by gcc differs between x86 and s390 or ppc
      gcc-4.7.2-8.fc18.s390
      gcc-4.7.2-7.fc18.ppc

      Version-Release number of selected component (if applicable):
      qpid-proton-0.3-1.fc18

        Activity

        Hide
        Dan Horák added a comment -

        It's still failing, my proposed patch was setting the "scanned" variable, your committed patch is setting the "found" variable. See log at http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=921511

        Show
        Dan Horák added a comment - It's still failing, my proposed patch was setting the "scanned" variable, your committed patch is setting the "found" variable. See log at http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=921511

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development