Tuesday, June 21, 2016

[389-commits] 2 commits - ldap/servers VERSION.sh

VERSION.sh | 2 +-
ldap/servers/plugins/acl/aclplugin.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 8b349638912f5ac33868ff91571ea15e3c0250d5
Author: Noriko Hosoi <nhosoi@redhat.com>
Date: Tue Jun 21 15:08:19 2016 -0700

bump version to 1.3.5.8

diff --git a/VERSION.sh b/VERSION.sh
index b9bbe37..f9222b2 100644
--- a/VERSION.sh
+++ b/VERSION.sh
@@ -10,7 +10,7 @@ vendor="389 Project"
# PACKAGE_VERSION is constructed from these
VERSION_MAJOR=1
VERSION_MINOR=3
-VERSION_MAINT=5.7
+VERSION_MAINT=5.8
# NOTE: VERSION_PREREL is automatically set for builds made out of a git tree
VERSION_PREREL=
VERSION_DATE=`date -u +%Y%m%d%H%M%S`


commit 4c689e77b4e5269baf4335d758d2ab66e50b43f3
Author: Noriko Hosoi <nhosoi@redhat.com>
Date: Tue Jun 21 15:04:09 2016 -0700

Ticket #48366 - proxyauth does not work bound as directory manager

Description: Coverity Defect type: RESOURCE_LEAK
aclplugin.c:131,151: leaked_storage: Variable "proxy_dn" going out
of scope leaks the storage it points to.

Even if proxy_dn is allocated, the return value of proxyauth_get_dn
could be other than LDAP_SUCCESS.

diff --git a/ldap/servers/plugins/acl/aclplugin.c b/ldap/servers/plugins/acl/aclplugin.c
index 50de2cc..f48b327 100644
--- a/ldap/servers/plugins/acl/aclplugin.c
+++ b/ldap/servers/plugins/acl/aclplugin.c
@@ -122,8 +122,8 @@ aclplugin_preop_search ( Slapi_PBlock *pb )

if (LDAP_SUCCESS == proxyauth_get_dn(pb, &proxy_dn, &errtxt) && proxy_dn) {
isProxy = 1;
- slapi_ch_free_string(&proxy_dn);
}
+ slapi_ch_free_string(&proxy_dn);

if ( isRoot && !isProxy) {
TNF_PROBE_1_DEBUG(aclplugin_preop_search_end ,"ACL","",

--
389-commits mailing list
389-commits@lists.fedoraproject.org
https://lists.fedoraproject.org/admin/lists/389-commits@lists.fedoraproject.org

No comments:

Post a Comment