[Date Prev][Date Next]
Re: (ITS#8158) cldap broken for aix and solaris
On 03/06/15 08:47, email@example.com wrote:
> Hmm, my fingers thought it was still february. Now I have uploaded a
> file with the correct name, goran-hammarback-150603.ext.
The date is probably just for disambiguation anyway.
Tweaked it a bit, I expect it should look like this:
Is there a reason you switched to sizeof(struct sockaddr) from
sockaddr_storage as the fallback? Otherwise sockaddr_storage
seems better, since that has presumably worked on most hosts.
Should only matter if people play tricks like using ldap_init_fd()
to create an unsupported connection type, e.g. "UDP ldapi://".
Anyway, please test (or protest if it should be sockaddr).
The change from your patch may be trivial, but that doesn't
mean I couldn't have managed to break it:-)
Note that OpenLDAP should be edited with editor tab width =
indentation = 4. Though some code has been written with
tab width = 8, so it looks messy either way.