[389-commits] ldap/servers

[Richard Allen Megginson]

 ldap/servers/slapd/pagedresults.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 0d4849dd7551347f0e24ac1027f4d0501084dcf3
Author: Rich Megginson <rmeggins at redhat.com>
Date:   Tue Dec 10 08:08:35 2013 -0700

    Ticket #47623 fix memleak caused by 47347
    
    https://fedorahosted.org/389/ticket/47623
    Reviewed by: tbordaz, nhosoi (Thanks!)
    Branch: master
    Fix Description: Create the mutex if it doesn't exist.
    Platforms tested: RHEL6 x86_64
    Flag Day: no
    Doc impact: no

diff --git a/ldap/servers/slapd/pagedresults.c b/ldap/servers/slapd/pagedresults.c
index a835d6b..9af5773 100644
--- a/ldap/servers/slapd/pagedresults.c
+++ b/ldap/servers/slapd/pagedresults.c
@@ -122,7 +122,6 @@ pagedresults_parse_control_value( Slapi_PBlock *pb,
                            sizeof(PagedResults) * maxlen);
             }
             *index = maxlen; /* the first position in the new area */
-            conn->c_pagedresults.prl_list[*index].pr_mutex = PR_NewLock();
         } else {
             for (i = 0; i < conn->c_pagedresults.prl_maxlen; i++) {
                 if (!conn->c_pagedresults.prl_list[i].pr_current_be) {
@@ -131,6 +130,9 @@ pagedresults_parse_control_value( Slapi_PBlock *pb,
                 }
             }
         }
+        if (!conn->c_pagedresults.prl_list[*index].pr_mutex) {
+            conn->c_pagedresults.prl_list[*index].pr_mutex = PR_NewLock();
+        }
         conn->c_pagedresults.prl_count++;
     } else {
         /* Repeated paged results request.