ITS#8890 fix benign typos

No functional impact

diff --git a/clients/tools/common.c b/clients/tools/common.c
index 3592e6fabc1f4980567f13d8fbe2faa9c1e5333b..5cefc54e3d35d075218efb7dd1cd8ac42bf1b0f6 100644 (file)
--- a/clients/tools/common.c
+++ b/clients/tools/common.c
@@ -2327,7 +2327,7 @@ void tool_print_ctrls(
                /* known controls */
                for ( j = 0; tool_ctrl_response[j].oid != NULL; j++ ) {
                        if ( strcmp( tool_ctrl_response[j].oid, ctrls[i]->ldctl_oid ) == 0 ) {
-                               if ( !tool_ctrl_response[j].mask & tool_type ) {
+                               if ( !(tool_ctrl_response[j].mask & tool_type )) {
                                        /* this control should not appear
                                         * with this tool; warning? */
                                }

diff --git a/servers/slapd/backend.c b/servers/slapd/backend.c
index e7a6f3b902ca8b893293dcf309356571868f641a..94bffaa6012b51b9a786bfb0400dfb62a6803e2b 100644 (file)
--- a/servers/slapd/backend.c
+++ b/servers/slapd/backend.c
@@ -1500,7 +1500,7 @@ fe_acl_group(
                                         * or if filter parsing fails.
                                         * In the latter case,
                                         * we should give up. */
-                                       if ( ludp->lud_filter != NULL && ludp->lud_filter != '\0') {
+                                       if ( ludp->lud_filter != NULL && *ludp->lud_filter != '\0') {
                                                filter = str2filter_x( op, ludp->lud_filter );
                                                if ( filter == NULL ) {
                                                        /* give up... */

diff --git a/servers/slapd/overlays/constraint.c b/servers/slapd/overlays/constraint.c
index 5c65b42dc90e07c3407ad11d12792e4287640833..ccc942f8df1ca9127c967097f97c839b06c7309c 100644 (file)
--- a/servers/slapd/overlays/constraint.c
+++ b/servers/slapd/overlays/constraint.c
@@ -446,7 +446,7 @@ constraint_cf_gen( ConfigArgs *c )
                                                }
 
                                                if ( ap.restrict_lud->lud_attrs != NULL ) {
-                                                       if ( ap.restrict_lud->lud_attrs[0] != '\0' ) {
+                                                       if ( ap.restrict_lud->lud_attrs[0] != NULL ) {
                                                                snprintf( c->cr_msg, sizeof( c->cr_msg ),
                                                                        "%s %s: attrs not allowed in restrict URI %s\n",
                                                                        c->argv[0], c->argv[1], arg);

diff --git a/servers/slapd/syntax.c b/servers/slapd/syntax.c
index 30cc81fb9b0187ae228c2d88c72e9edc522fdde0..afa9f36041e21eef9de5923be92391ce7d1d911c 100644 (file)
--- a/servers/slapd/syntax.c
+++ b/servers/slapd/syntax.c
@@ -219,8 +219,8 @@ syn_add(
                        }
 
                        assert( (*lsei)->lsei_values != NULL );
-                       if ( (*lsei)->lsei_values[0] == '\0'
-                               || (*lsei)->lsei_values[1] != '\0' )
+                       if ( (*lsei)->lsei_values[0] == NULL
+                               || (*lsei)->lsei_values[1] != NULL )
                        {
                                Debug( LDAP_DEBUG_ANY, "syn_add(%s): exactly one substitute syntax must be present\n",
                                        ssyn->ssyn_syn.syn_oid, 0, 0 );