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.

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

Minor bug in error reporting ldap connection on server side #10

Closed
copumpkin opened this issue Feb 23, 2015 · 2 comments
Closed

Minor bug in error reporting ldap connection on server side #10

copumpkin opened this issue Feb 23, 2015 · 2 comments

Comments

@copumpkin
Copy link
Contributor

    bindErr := ldapServer.Bind(config.LDAP.Bind.DN, config.LDAP.Bind.Password)
    if bindErr != nil {
        log.Error("Could not bind to LDAP! %s", err.Error())

The Bind call is saved to bindErr, but the error message attempts to print err, which at that point will be nil. This leads to a nil pointer dereference error instead of the actual desired error message 😄

@frangarciam
Copy link
Contributor

Opened #17

Thanks for all the bug reports!

@zylad
Copy link
Contributor

zylad commented Mar 10, 2015

Merged, thanks.

@zylad zylad closed this as completed Mar 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants