diff options
author | Rich Megginson <rmeggins@redhat.com> | 2005-03-24 17:12:06 +0000 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2005-03-24 17:12:06 +0000 |
commit | b6bff935c117b5042cd2a9809218607c01bea965 (patch) | |
tree | ff24a822b63d879254a679bad694b09d2454bc67 /ldap/servers/plugins/distrib | |
parent | d80d9cceedbe18c97d240fca0ee6c20d01253423 (diff) | |
download | ds-b6bff935c117b5042cd2a9809218607c01bea965.tar.gz ds-b6bff935c117b5042cd2a9809218607c01bea965.tar.xz ds-b6bff935c117b5042cd2a9809218607c01bea965.zip |
Bug(s) fixed: 152030
Bug Description: Various valgrind reported problems. Mostly not serious, but a few which could cause bad leaks in certain situations.
Reviewed by: Rob and David (Thanks!)
Fix Description: Mostly added frees where needed, or moved things around to make sure the proper free was called. I also fixed the formatting/spacing.
Platforms tested: RHEL3
Flag Day: no
Doc impact: No.
QA impact: should be covered by regular nightly and manual testing
New Tests integrated into TET: none
Diffstat (limited to 'ldap/servers/plugins/distrib')
-rw-r--r-- | ldap/servers/plugins/distrib/distrib.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ldap/servers/plugins/distrib/distrib.c b/ldap/servers/plugins/distrib/distrib.c index 8b40a06a..6baf1cea 100644 --- a/ldap/servers/plugins/distrib/distrib.c +++ b/ldap/servers/plugins/distrib/distrib.c @@ -74,6 +74,7 @@ int alpha_distribution(Slapi_PBlock *pb, Slapi_DN * target_dn, slapi_sdn_get_rdn(target_dn, rdn); slapi_rdn_get_first(rdn, &rdn_type, &rdn_value); c = rdn_value[0]; + slapi_rdn_free(&rdn); if (!(((c >= 'a') && (c <= 'z')) || ((c >= 'A') && (c <= 'Z')) )) @@ -81,7 +82,6 @@ int alpha_distribution(Slapi_PBlock *pb, Slapi_DN * target_dn, return 0; } - slapi_rdn_free(&rdn); /* for entries with rdn starting with alphabetic characters * use the formula : (c - 'A') * be_count/26 |