Uploaded image for project: 'Couchbase Lite'
  1. Couchbase Lite
  2. CBL-582

Memory leak of FLDoc

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.7.0
    • 2.7.1
    • iOS
    • Security Level: Public
    • None
    • 2

    Description

      There is an unreleased `FLDoc *` instance here and here

      Every time either of these functions are called they will leak FLDoc * instances

      Attachments

        Issue Links

          relates to

          Task - A task that needs to be done. CBL-540 Memory Leaks

          • Major - Major loss of function.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            Ascending order - Click to sort in descending order
            Jayahari.Vavachan Jay Vavachan added a comment - - edited

            FLDoc is same as Doc, and Doc is RefCounted (here)

            But I can see the FLSliceResult might need to be released in the same scope.

            Related PRs
            https://github.com/couchbase/couchbase-lite-ios/pull/2576
            https://github.com/couchbase/couchbase-lite-ios/pull/2581

            Jayahari.Vavachan Jay Vavachan added a comment - - edited FLDoc is same as Doc, and Doc is RefCounted ( here ) But I can see the FLSliceResult might need to be released in the same scope. Related PRs https://github.com/couchbase/couchbase-lite-ios/pull/2576 https://github.com/couchbase/couchbase-lite-ios/pull/2581
            build-team Couchbase Build Team added a comment -

            Build couchbase-lite-ios-2.8.0-6 contains couchbase-lite-ios commit befa595 with commit message:
            CBL-582: Fix: FLDoc memory fix (#2596)

            build-team Couchbase Build Team added a comment - Build couchbase-lite-ios-2.8.0-6 contains couchbase-lite-ios commit befa595 with commit message: CBL-582 : Fix: FLDoc memory fix (#2596)

            People

              Jayahari.Vavachan Jay Vavachan
              jimb Jim Borden
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty