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/slapd/pw_mgmt.c | |
| 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/slapd/pw_mgmt.c')
| -rw-r--r-- | ldap/servers/slapd/pw_mgmt.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ldap/servers/slapd/pw_mgmt.c b/ldap/servers/slapd/pw_mgmt.c index def09c02..ee102159 100644 --- a/ldap/servers/slapd/pw_mgmt.c +++ b/ldap/servers/slapd/pw_mgmt.c @@ -121,7 +121,8 @@ skip: pw_exp_date != NOT_FIRST_TIME && (diff_t = difftime ( pw_exp_date, parse_genTime ( cur_time_str ))) <= 0 ) { - + + slapi_ch_free_string(&cur_time_str); /* only need this above */ /* password has expired. Check the value of * passwordGraceUserTime and compare it * against the value of passwordGraceLimit */ @@ -175,7 +176,6 @@ skip: /* Apply current modifications */ pw_apply_mods(dn, &smods); slapi_mods_done(&smods); - slapi_ch_free((void **) &cur_time_str ); delete_passwdPolicy(&pwpolicy); return (-1); } |