Merge pull request #244 from olivierbeytrison/rlm_ldap_fix

fix warnings in ldap.c
Fix typo
Use pairmove2() instead of foo->vps = bar->vps;bar->vps = NULL;

There are probably other instances of this in the code
Call talloc_zero, not talloc in ttls

And remove an unused function in EAP-MD5
Logintime no longer removes all reply messages if user is locked out (this was wrong)

Logintime no longer sets a reply message, this can be done from unlang, and so this is duplicative functionality

Cleanup logintime code