Mailing-List: contact commits-help@apr.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@apr.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1441742 - /apr/apr/trunk/poll/unix/z_asio.c
Date: Sat, 02 Feb 2013 12:50:59 -0000
To: commits@apr.apache.org
From: gregames@apache.org
Message-Id: <20130202125059.1C88123888CD@eris.apache.org>

Author: gregames
Date: Sat Feb  2 12:50:58 2013
New Revision: 1441742

URL: http://svn.apache.org/viewvc?rev=1441742&view=rev
Log:
fix memory leak

Modified:
    apr/apr/trunk/poll/unix/z_asio.c

Modified: apr/apr/trunk/poll/unix/z_asio.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/poll/unix/z_asio.c?rev=1441742&r1=1441741&r2=1441742&view=diff
==============================================================================
--- apr/apr/trunk/poll/unix/z_asio.c (original)
+++ apr/apr/trunk/poll/unix/z_asio.c Sat Feb  2 12:50:58 2013
@@ -63,7 +63,7 @@ struct apr_pollset_private_t
     apr_thread_mutex_t *ring_lock;
 #endif
 
-    /* A hash of all active elements used for O(1) garbage collection */
+    /* A hash of all active elements used for O(1) _remove operations */
     apr_hash_t      *elems;
 
     APR_RING_HEAD(ready_ring_t,       asio_elem_t)      ready_ring;
@@ -640,6 +640,15 @@ static apr_status_t asio_pollset_poll(ap
          */
 
         if (elem->state == ASIO_REMOVED) {
+
+            /* 
+             * async i/o is done since it was found on prior_ready
+             * the state says the caller is done with it too 
+             * so recycle the elem 
+             */
+             
+            APR_RING_INSERT_TAIL(&(priv->free_ring), elem,
+                                 asio_elem_t, link);
             continue;  /* do not re-add if it has been _removed */
         }