#643 Don't pass NULL to printf for TLS errors
Closed: Fixed None Opened 13 years ago by sejeff.

<sbose> SEJeff: thanks for catching this. Can you file a bug for the 'LDAP connection error: (null)'. It is safe to pass NULL to printf in glibc but might not be so in other libcs.

The way I caused this was by futzing the sssd.conf and pointing ldap_uri to hosts that don't run ldap. After restarting sssd (which came up in offline mode), I tried to change the password. It printed these errors:

<27> Oct 12 07:48:45.000 ops2.dev1.int sssd[be[LDAP]]: Could not start TLS. (null)
<27> Oct 12 07:48:45.000 ops2.dev1.int sssd[be[LDAP]]: Could not start TLS. (null)
<27> Oct 12 07:48:45.000 ops2.dev1.int sssd[be[LDAP]]: Could not start TLS. (null)

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.6.0

Fields changed

coverity: =>
owner: somebody => jhrozek
upgrade: => 0

Fixed in 4a28fb1

resolution: => fixed
status: new => closed

Backported to sssd-1-5

b6ed1d6

milestone: SSSD 1.6.0 => SSSD 1.5.9
patch: => 0
rhbz: =>

Fields changed

rhbz: => 0

Metadata Update from @sejeff:
- Issue assigned to jhrozek
- Issue set to the milestone: SSSD 1.5.9

7 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/1685

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata