ldb_ldap: fix off-by-one increment in lldb_add_msg_attr
authorAlexander Bokovoy <ab@samba.org>
Thu, 18 Jun 2020 07:45:41 +0000 (10:45 +0300)
committerAndreas Schneider <asn@cryptomilk.org>
Fri, 19 Jun 2020 08:35:33 +0000 (08:35 +0000)
commit990a0fc4a0481aed817fad7575d8df453fbe7af9
tree5160e61955048ef54bd307a59b773fafeaf315b0
parent36bd6edd8a603f3aa34aff81c43ef26efd3ad4cf
ldb_ldap: fix off-by-one increment in lldb_add_msg_attr

Fix regression introduced by commit ce2bf5c72b6423fff680b3d6a9042103a6cdda55

lldb_add_msg_attr() calls ldb_msg_add_empty() which, in turn, calls
calls _ldb_msg_add_el() which already increments msg->num_elements by one.

As a result, msg->num_elements is bigger than the actual number of
elements and any iteration over elements would step over elements array
boundary.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14413
Signed-off-by: Alexander Bokovoy <ab@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org>
Autobuild-Date(master): Fri Jun 19 08:35:33 UTC 2020 on sn-devel-184
lib/ldb/ldb_ldap/ldb_ldap.c