diff options
| author | Noriko Hosoi <nhosoi@redhat.com> | 2008-04-18 20:18:59 +0000 |
|---|---|---|
| committer | Noriko Hosoi <nhosoi@redhat.com> | 2008-04-18 20:18:59 +0000 |
| commit | 1580bebf61d090c01ffdee71d9a17551106604c5 (patch) | |
| tree | c0f8694901a137d42560007613467e70932fe21f | |
| parent | 4309122620a3266b4ba72d6b556a35414e02a9d1 (diff) | |
| download | ds-1580bebf61d090c01ffdee71d9a17551106604c5.tar.gz ds-1580bebf61d090c01ffdee71d9a17551106604c5.tar.xz ds-1580bebf61d090c01ffdee71d9a17551106604c5.zip | |
Resolves: #428764Directory_Server_8_0_Errata_2008-0265
Summary: memory leaks in extensible filter code
Description: applying the patch provided by Ulf Weltman
1) type is not consumed in attrlist_merge
2) although dnattrs is a linked list, only the first item was released.
Test case filter: "(ou:dn:=groups)"
| -rw-r--r-- | ldap/servers/slapd/filterentry.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/ldap/servers/slapd/filterentry.c b/ldap/servers/slapd/filterentry.c index 317c4f24..1e7ab8bb 100644 --- a/ldap/servers/slapd/filterentry.c +++ b/ldap/servers/slapd/filterentry.c @@ -441,6 +441,7 @@ dn2attrs(const char *dn) bv.bv_len = strlen(val); bvec[0] = &bv; attrlist_merge (&dnAttrs, type, bvec); + slapi_ch_free_string( &type ); } } ldap_value_free (avas); @@ -505,7 +506,7 @@ test_extensible_filter( /* B) Also check the DN attributes for the attribute value */ Slapi_Attr* dnattrs= dn2attrs(slapi_entry_get_dn_const(e)); rc= test_ava_filter( callers_pb, e, dnattrs, &a, LDAP_FILTER_EQUALITY, 0 /* Don't Verify Access */ , 0 /* don't just verify access */, access_check_done ); - slapi_attr_free( &dnattrs ); + attrlist_free( dnattrs ); } } else @@ -520,7 +521,7 @@ test_extensible_filter( /* D & F) Also check the DN attributes for the attribute value */ Slapi_Attr* dnattrs= dn2attrs(slapi_entry_get_dn_const(e)); mrf->mrf_match (mrf->mrf_object, e, dnattrs); - slapi_attr_free( &dnattrs ); + attrlist_free( dnattrs ); } } } @@ -573,7 +574,7 @@ test_extensible_filter( /* B) Also check the DN attributes for the attribute value */ Slapi_Attr* dnattrs= dn2attrs(slapi_entry_get_dn_const(e)); rc= test_ava_filter( callers_pb, e, dnattrs, &a, LDAP_FILTER_EQUALITY, 0 /* Don't Verify Access */ , 0 /* don't just verify access */, access_check_done ); - slapi_attr_free( &dnattrs ); + attrlist_free( dnattrs ); } } else @@ -588,7 +589,7 @@ test_extensible_filter( /* D & F) Also check the DN attributes for the attribute value */ Slapi_Attr* dnattrs= dn2attrs(slapi_entry_get_dn_const(e)); mrf->mrf_match (mrf->mrf_object, e, dnattrs); - slapi_attr_free( &dnattrs ); + attrlist_free( dnattrs ); } } } |