On Sat, Jun 30, 2018 at 03:07:07PM +0100, Stuart Henderson wrote: > On 2018/06/29 09:18, Rob Pierce wrote: > > CVSROOT: /cvs > > Module name: src > > Changes by: r...@cvs.openbsd.org 2018/06/29 09:18:03 > > > > Modified files: > > usr.bin/ldap : ber.c ber.h > > usr.sbin/ldapd : ber.c > > usr.sbin/ypldap: ber.c ber.h > > usr.sbin/snmpd : ber.3 ber.c > > > > Log message: > > Synchronize ber.c and ber.h across ldap, ldapd, and ypldap, and reduce diff > > with snmpd. More tweaks to come once things are fully synchronized. > > > > Feedback from claudio and Robert Klein. > > > > Ok claudio@ > > > > I've reverted the snmpd part of this, it breaks SNMPv3 authentication: > > snmpwalk: Authentication failure (incorrect password, community or key) > > If you don't have an easy way to test yourself I'm happy to test things > if you send me a diff.
I just saw your commit. Sorry about that. Yes - I will send you any subsequent diffs for testing. Much appreciated. This is good though, because we have a divergence in ber.c that works for some comsumers and not for others. Hopefully we can fix that. I am looking at that one change now to see what is going on. Thank you for your email. Regards, Rob