diff options
author | Rich Megginson <rmeggins@redhat.com> | 2010-10-01 16:38:04 -0600 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2010-10-01 17:01:57 -0600 |
commit | 4f410d762b008da8e2e43e29100c2c04ff332fbb (patch) | |
tree | 6723ef832b034bda64cc691d252b42e522718bfa /ldap/servers/plugins/replication/repl5_total.c | |
parent | a82f61269d69882d19b15def70de68906ed5740e (diff) | |
download | ds-4f410d762b008da8e2e43e29100c2c04ff332fbb.tar.gz ds-4f410d762b008da8e2e43e29100c2c04ff332fbb.tar.xz ds-4f410d762b008da8e2e43e29100c2c04ff332fbb.zip |
openldap ber_init will assert if the bv->bv_val is NULL
Have to ensure that all usage of ber_init in the server checks to see if
the bv->bv_val is non-NULL before using ber_init, and return the appropriate
error if it is NULL
Also fixed a problem in dna_extend_exop - would not send the ldap result to
the client in certain error conditions
Reviewed by: nhosoi (Thanks!)
Tested on: RHEL5 x86_64
Diffstat (limited to 'ldap/servers/plugins/replication/repl5_total.c')
-rw-r--r-- | ldap/servers/plugins/replication/repl5_total.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ldap/servers/plugins/replication/repl5_total.c b/ldap/servers/plugins/replication/repl5_total.c index d2987cdb..99ba838a 100644 --- a/ldap/servers/plugins/replication/repl5_total.c +++ b/ldap/servers/plugins/replication/repl5_total.c @@ -729,7 +729,7 @@ decode_total_update_extop(Slapi_PBlock *pb, Slapi_Entry **ep) if (NULL == extop_oid || ((strcmp(extop_oid, REPL_NSDS50_REPLICATION_ENTRY_REQUEST_OID) != 0) && (strcmp(extop_oid, REPL_NSDS71_REPLICATION_ENTRY_REQUEST_OID) != 0)) || - NULL == extop_value) + NULL == extop_value || NULL == extop_value->bv_val) { /* Bogus */ goto loser; |