Uploaded image for project: 'ZooKeeper'
  1. ZooKeeper
  2. ZOOKEEPER-1632

fix memory leaks in cli_st

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 3.4.6, 3.5.0
    • c client
    • None
    • The fix for this issue solves the memory leak spotted in the absence of errors. In the case the completion function is not registered because of an error (e.g., see zoo_async), the line duplicate won't be freed.

    Description

      Fix two memory leaks revealed by running:

      valgrind --leak-check=full ./.libs/cli_st 127.0.0.1:2182
create /foo
quit

      Attachments

        1. ZOOKEEPER-1632.001.patch
          2 kB
          Colin McCabe
        2. ZOOKEEPER-1632.002.patch
          2 kB
          Colin McCabe
        3. ZOOKEEPER-1632-b3.4.patch
          1 kB
          Flavio Paiva Junqueira
        4. ZOOKEEPER-1632-trunk.patch
          2 kB
          Flavio Paiva Junqueira

        Issue Links

        relates to

        Bug - A problem which impairs or prevents the functions of the product. ZOOKEEPER-1556 Memory leak reported by valgrind mt version

        • Minor - Minor loss of function, or other problem where easy workaround is present.
        • Open

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            fpj Flavio Paiva Junqueira
            cmccabe Colin McCabe
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment