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

Segfault in broker while processing active bridges

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Critical
    • Resolution: Fixed
    • qpid-cpp-1.36.0
    • qpid-cpp-1.37.0
    • C++ Broker
    • None
    • Ubuntu 17.10 x86_64, gcc 7.

    • Important

    Description

      Segfault occurs on a brackground thread within about 5-10 seconds of broker startup at src/qpid/broker/Link.cpp:465. segfault stack trace attached, frames #3 and #5 are of particular relevance.
      The unchecked Bridge::shared_ptr derived from the iterator is null and the invocation of bridge->closed() triggers the segfault. Adding a simple null check (as per attached segfault-fix.patch) fixes the segfault but not the underlying reason for the null pointer.

      The segfault appears to be related to how a second broker (henceforth "broker1") is configured; this is the one to which the links are established. Without broker1, the "segfaulting broker" (aka "broker2") does not do its thing. It may be that broker1 returns invalid data to broker2 but this is not in the scope of this bug report, which focuses on the segfault.

      Reproduce

      Unfortunately the steps to arrive at this situation are not clear so the reproduce is a bit hacky - the data directory, config file and some certs for the two brokers are attached as a tarball in the hope that they can be arranged in such a way as to provide a reproduce in lieu of a purely step-based procedure.
      Steps to reproduce:

      • Temporarily add a DNS alias to the local machine of "octopussy" (necessary due to cert config and durable link config in broker2's data store)
      • Extract the attached segfault-repoduce.tar.gz to an empty directory (assumed to be cwd)
      • Start broker1 with "qpidd --config broker1/qpidd.conf"
      • In another shell with the same cwd, start broker2 with "qpidd --config broker2/qpidd.conf"
      • Observe segfault in broker2 after 5-10 seconds.

      Attachments

        1. segfault stack trace
          19 kB
          Chris Richardson
        2. segfault-repoduce.tar.gz
          263 kB
          Chris Richardson
        3. segfault-fix.patch
          0.8 kB
          Chris Richardson
        4. std_remove_if_with_smart_ptr.cpp
          1 kB
          Chris Richardson

        Activity

          People

            aconway Alan Conway
            chris.richardson Chris Richardson
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 48h
                48h
                Remaining:
                Remaining Estimate - 48h
                48h
                Logged:
                Time Spent - Not Specified
                Not Specified