featInfo.ldapaif_name free check in ldap_get_option

KxSystems / ldap

Kdb+ integration with LDAP

https://code.kx.com/q

Apache License 2.0

5 stars 5 forks source link

Closed sshanks-kx closed 4 years ago

sshanks-kx commented 4 years ago

Does ldap_get_option reallocate featInfo.ldapaif_name - would it need to be freed? Need to check

sshanks-kx commented 4 years ago

@nmcdonnell-kx

Looks like already free'd

featInfo.ldapaif_name = info.ldapai_extensions[i];
....
ldap_memfree(info.ldapai_extensions[i]);

sshanks-kx commented 4 years ago

ldap source code has similar action in libraries/libldap/apitest.c