From 6946e494d6305a8fc140f509de146101cdaacab0 Mon Sep 17 00:00:00 2001 From: rob Date: Wed, 4 Jul 2018 13:04:30 +0000 Subject: Relocate the update of br_offs from ber_read() to ber_readbuf() so ber_getc() can call ber_readbuf() in all cases. This resolves a problem previously encountered with SNMPv3 authentication, simplifies the code, and completes a full synchronization of all ber instances. Proposed by claudio@. Problematic use case in snmpd tested by sthen@ and me. ldap(s) appear happy as well. looks good to claudio@ --- usr.sbin/ldapd/ber.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) (limited to 'usr.sbin/ldapd') diff --git a/usr.sbin/ldapd/ber.c b/usr.sbin/ldapd/ber.c index 94babe9693e..c2e15057870 100644 --- a/usr.sbin/ldapd/ber.c +++ b/usr.sbin/ldapd/ber.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ber.c,v 1.18 2018/07/03 18:49:10 rob Exp $ */ +/* $OpenBSD: ber.c,v 1.19 2018/07/04 13:04:30 rob Exp $ */ /* * Copyright (c) 2007, 2012 Reyk Floeter @@ -1212,8 +1212,7 @@ ber_read_element(struct ber *ber, struct ber_element *elm) static ssize_t ber_readbuf(struct ber *b, void *buf, size_t nbytes) { - size_t sz; - size_t len; + size_t sz, len; if (b->br_rbuf == NULL) return -1; @@ -1227,6 +1226,7 @@ ber_readbuf(struct ber *b, void *buf, size_t nbytes) bcopy(b->br_rptr, buf, len); b->br_rptr += len; + b->br_offs += len; return (len); } @@ -1284,14 +1284,10 @@ ber_read(struct ber *ber, void *buf, size_t len) r = ber_readbuf(ber, b, remain); if (r == -1) return -1; - if (r == 0) - return (b - (u_char *)buf); b += r; remain -= r; } - r = b - (u_char *)buf; - ber->br_offs += r; - return r; + return (b - (u_char *)buf); } int -- cgit v1.2.3-59-g8ed1b