zrhoffman / rpms / 389-ds-base

Forked from rpms/389-ds-base 3 years ago
Clone

Blame SOURCES/0074-Ticket-47623-fix-memleak-caused-by-47347.patch

cc3dff
From b910f02f74410f44d0285d77b15e5d399df3c1a8 Mon Sep 17 00:00:00 2001
cc3dff
From: Rich Megginson <rmeggins@redhat.com>
cc3dff
Date: Mon, 9 Dec 2013 17:00:32 -0700
cc3dff
Subject: [PATCH 74/78] Ticket #47623 fix memleak caused by 47347
cc3dff
cc3dff
https://fedorahosted.org/389/ticket/47623
cc3dff
Reviewed by: nhosoi (Thanks!)
cc3dff
Branch: 389-ds-base-1.3.1
cc3dff
Fix Description: Only need to create the mutex when creating a new PR object.
cc3dff
Platforms tested: RHEL6 x86_64
cc3dff
Flag Day: no
cc3dff
Doc impact: no
cc3dff
(cherry picked from commit 98ccb602058270e97a3702ae2b81c17635af8d27)
cc3dff
(cherry picked from commit 65c51555c0ecc94c5d93f09124168697ba1db6b3)
cc3dff
(cherry picked from commit 8a2c666df491b7c8666f8a70a5038b35c43fbc3b)
cc3dff
---
cc3dff
 ldap/servers/slapd/pagedresults.c | 2 +-
cc3dff
 1 file changed, 1 insertion(+), 1 deletion(-)
cc3dff
cc3dff
diff --git a/ldap/servers/slapd/pagedresults.c b/ldap/servers/slapd/pagedresults.c
cc3dff
index 78bd6b0..a835d6b 100644
cc3dff
--- a/ldap/servers/slapd/pagedresults.c
cc3dff
+++ b/ldap/servers/slapd/pagedresults.c
cc3dff
@@ -122,6 +122,7 @@ pagedresults_parse_control_value( Slapi_PBlock *pb,
cc3dff
                            sizeof(PagedResults) * maxlen);
cc3dff
             }
cc3dff
             *index = maxlen; /* the first position in the new area */
cc3dff
+            conn->c_pagedresults.prl_list[*index].pr_mutex = PR_NewLock();
cc3dff
         } else {
cc3dff
             for (i = 0; i < conn->c_pagedresults.prl_maxlen; i++) {
cc3dff
                 if (!conn->c_pagedresults.prl_list[i].pr_current_be) {
cc3dff
@@ -131,7 +132,6 @@ pagedresults_parse_control_value( Slapi_PBlock *pb,
cc3dff
             }
cc3dff
         }
cc3dff
         conn->c_pagedresults.prl_count++;
cc3dff
-        conn->c_pagedresults.prl_list[*index].pr_mutex = PR_NewLock();
cc3dff
     } else {
cc3dff
         /* Repeated paged results request.
cc3dff
          * PagedResults is already allocated. */
cc3dff
-- 
cc3dff
1.8.1.4
cc3dff