Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.12, 0.18
    • Fix Version/s: 0.19
    • Component/s: C++ Broker, C++ Client
    • Labels:
      None
    • Environment:

      Windows

      Description

      In our large system performance test environments we see Broker memory growth until the process crashes. This appears to be reproducible simply by opening and closing connections as fast as possible. Marked as 'Blocker' as it was blocking shipment of our Windows based product - the Broker would exhaust memory and crash itself every few days of operation.

        Activity

        Hide
        Cliff Jansen added a comment -

        fixed r1397926.

        Show
        Cliff Jansen added a comment - fixed r1397926.
        Hide
        Cliff Jansen added a comment -

        Nice.

        Hats off to you gentlemen. I will apply the patch later this week if no objections pop-up.

        Show
        Cliff Jansen added a comment - Nice. Hats off to you gentlemen. I will apply the patch later this week if no objections pop-up.
        Hide
        Kerry Bonin added a comment -

        Fix for Windows memory leak. One of our team leads, Ben Holm, developed this fix. It appears to resolve the leak in small scale testing, we are about to test it in our large scale environment. This appears to be an improperly handled corner case, where (at least on Windows) one buffer is leaked per connection close. This fix hands the buffer back to a managed pool.

        Show
        Kerry Bonin added a comment - Fix for Windows memory leak. One of our team leads, Ben Holm, developed this fix. It appears to resolve the leak in small scale testing, we are about to test it in our large scale environment. This appears to be an improperly handled corner case, where (at least on Windows) one buffer is leaked per connection close. This fix hands the buffer back to a managed pool.

          People

          • Assignee:
            Cliff Jansen
            Reporter:
            Kerry Bonin
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development