Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ldap: Stop using wide char version of ldapp_err2string #4272

Closed
wants to merge 1 commit into from
Closed

ldap: Stop using wide char version of ldapp_err2string #4272

wants to merge 1 commit into from

Commits on Sep 9, 2019

  1. ldap: Stop using wide char version of ldapp_err2string 

    Despite ldapp_err2string being documented by MS as returning a
PCHAR (char *), when UNICODE it is mapped to ldap_err2stringW and
returns PWCHAR (wchar_t *).

We have lots of code like that expects ldap_err2string to return char *,
most of it failf used like this:

failf(data, "LDAP local: Some error: %s", ldap_err2string(rc));

Closes #xxxx
    @jay
    jay committed Sep 9, 2019
    Configuration menu
    Copy the full SHA
    853822f View commit details
    Browse the repository at this point in the history