Happy New Year

[openldap] / servers / slapd / overlays / unique.c

diff --git a/servers/slapd/overlays/unique.c b/servers/slapd/overlays/unique.c
index c6bb780d7645a4f0132b7014636f6ce06641ffde..ed62d03b8938d5e935f77f9ce44f4e0db97505a4 100644 (file)
--- a/servers/slapd/overlays/unique.c
+++ b/servers/slapd/overlays/unique.c
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2004-2015 The OpenLDAP Foundation.
+ * Copyright 2004-2018 The OpenLDAP Foundation.
  * Portions Copyright 2004,2006-2007 Symas Corporation.
  * All rights reserved.
  *
@@ -1170,6 +1170,11 @@ unique_modify(
        Debug(LDAP_DEBUG_TRACE, "==> unique_modify <%s>\n",
              op->o_req_dn.bv_val, 0, 0);
 
+       if ( !op->orm_modlist ) {
+               Debug(LDAP_DEBUG_TRACE, "unique_modify: got empty modify op\n", 0, 0, 0);
+               return rc;
+       }
+
        /* skip the checks if the operation has manageDsaIt control in it
         * (for replication) */
        if ( op->o_managedsait > SLAP_CONTROL_IGNORED
@@ -1203,22 +1208,14 @@ unique_modify(
                             && !dnIsSuffix( &op->o_req_ndn, &uri->ndn ))
                                continue;
 
-                       if ( !(m = op->orm_modlist) ) {
-                               op->o_bd->bd_info = (BackendInfo *) on->on_info;
-                               send_ldap_error(op, rs, LDAP_INVALID_SYNTAX,
-                                               "unique_modify() got null op.orm_modlist");
-                               rc = rs->sr_err;
-                               break;
-
-                       } else
-                               for ( ; m; m = m->sml_next)
-                                       if ( (m->sml_op & LDAP_MOD_OP)
-                                            != LDAP_MOD_DELETE )
-                                               ks += count_filter_len
-                                                       ( domain,
-                                                         uri,
-                                                         m->sml_desc,
-                                                         m->sml_values);
+                       for ( m = op->orm_modlist; m; m = m->sml_next)
+                               if ( (m->sml_op & LDAP_MOD_OP)
+                                    != LDAP_MOD_DELETE )
+                                       ks += count_filter_len
+                                               ( domain,
+                                                 uri,
+                                                 m->sml_desc,
+                                                 m->sml_values);
 
                        /* skip this domain-uri if it isn't involved */
                        if ( !ks ) continue;