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

        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