diff options
| author | Nathan Kinder <nkinder@redhat.com> | 2010-09-08 14:16:38 -0700 |
|---|---|---|
| committer | Nathan Kinder <nkinder@redhat.com> | 2010-09-09 13:40:36 -0700 |
| commit | 6be23ba8b516f965a7ddadecbf9d25c513085d32 (patch) | |
| tree | 54341c27c9bfead9af0aaff5391d49f69aa92547 /ldap | |
| parent | d02e95349a038ca3f096b5432c8604b8ca478e1c (diff) | |
| download | ds-6be23ba8b516f965a7ddadecbf9d25c513085d32.tar.gz ds-6be23ba8b516f965a7ddadecbf9d25c513085d32.tar.xz ds-6be23ba8b516f965a7ddadecbf9d25c513085d32.zip | |
Bug 630094 - (cov#15451) Get rid of unreachable free statements
We need to remove the last "if (dnParts)" condition since it will
never be true.
The last frees of newDN, sval, and newvalue are also unnecessary
since they are only set in the non subtree rename case, where they
are already freed as well.
Diffstat (limited to 'ldap')
| -rw-r--r-- | ldap/servers/plugins/referint/referint.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/ldap/servers/plugins/referint/referint.c b/ldap/servers/plugins/referint/referint.c index 3ef9de6b..32249e93 100644 --- a/ldap/servers/plugins/referint/referint.c +++ b/ldap/servers/plugins/referint/referint.c @@ -490,15 +490,6 @@ _update_one_per_mod(const char *entryDN, /* DN of the searched entry */ slapi_ch_free_string(&newDN); } - /* in case these memories have not freed */ - slapi_ch_free_string(&newvalue); - slapi_ch_free_string(&sval); - if (dnParts){ - slapi_ldap_value_free(dnParts); - dnParts = NULL; - } - slapi_ch_free_string(&newDN); - return rc; } |