Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.4.3
    • Fix Version/s: 3.4.4, 3.5.0
    • Component/s: c client
    • Labels:
      None
    • Environment:

      windows7

      Description

      At line 99 in winport.c, use windows API "InitializeCriticalSection" but never call "DeleteCriticalSection"

        Activity

        Hide
        brooklin added a comment -

        fix memory when InitialCriticalSection but not Delete issue.

        Show
        brooklin added a comment - fix memory when InitialCriticalSection but not Delete issue.
        Hide
        Michi Mutsuzaki added a comment -

        +1.

        Thank you for the patch, brooklin!

        --Michi

        Show
        Michi Mutsuzaki added a comment - +1. Thank you for the patch, brooklin! --Michi
        Hide
        Mahadev konar added a comment -

        Michi is this committed to 3.4 branch as well?

        Show
        Mahadev konar added a comment - Michi is this committed to 3.4 branch as well?
        Hide
        Michi Mutsuzaki added a comment -

        I'm committing this to 3.4 branch right now.

        --Michi

        Show
        Michi Mutsuzaki added a comment - I'm committing this to 3.4 branch right now. --Michi
        Hide
        Hudson added a comment -

        Integrated in ZooKeeper-trunk #1665 (See https://builds.apache.org/job/ZooKeeper-trunk/1665/)
        ZOOKEEPER-1536 c client : memory leak in winport.c (brooklin via michim) (Revision 1379046)

        Result = SUCCESS
        michim : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1379046
        Files :

        • /zookeeper/trunk/CHANGES.txt
        • /zookeeper/trunk/src/c/src/winport.c
        Show
        Hudson added a comment - Integrated in ZooKeeper-trunk #1665 (See https://builds.apache.org/job/ZooKeeper-trunk/1665/ ) ZOOKEEPER-1536 c client : memory leak in winport.c (brooklin via michim) (Revision 1379046) Result = SUCCESS michim : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1379046 Files : /zookeeper/trunk/CHANGES.txt /zookeeper/trunk/src/c/src/winport.c

          People

          • Assignee:
            brooklin
            Reporter:
            brooklin
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development