s4:dsdb/schema: enforce rangeLower/nextUpper with WERR_DS_CONSTRAINT_VIOLATION

Signed-off-by: Stefan Metzmacher <metze@samba.org>

diff --git a/source4/dsdb/schema/schema_syntax.c b/source4/dsdb/schema/schema_syntax.c
index f7176cd1898e91d9d89c0e4fca251342106bdee0..2752645e80c60d1f5b8f2cd9e732de86ac9fd3d0 100644 (file)
--- a/source4/dsdb/schema/schema_syntax.c
+++ b/source4/dsdb/schema/schema_syntax.c
@@ -389,13 +389,13 @@ static WERROR dsdb_syntax_INT32_validate_ldb(const struct dsdb_syntax_ctx *ctx,
 
                if (attr->rangeLower) {
                        if ((int32_t)v < (int32_t)*attr->rangeLower) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
 
                if (attr->rangeUpper) {
                        if ((int32_t)v > (int32_t)*attr->rangeUpper) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
        }
@@ -514,13 +514,13 @@ static WERROR dsdb_syntax_INT64_validate_ldb(const struct dsdb_syntax_ctx *ctx,
 
                if (attr->rangeLower) {
                        if ((int64_t)v < (int64_t)*attr->rangeLower) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
 
                if (attr->rangeUpper) {
                        if ((int64_t)v > (int64_t)*attr->rangeUpper) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
        }
@@ -657,13 +657,13 @@ static WERROR dsdb_syntax_NTTIME_UTC_validate_ldb(const struct dsdb_syntax_ctx *
 
                if (attr->rangeLower) {
                        if ((int32_t)t < (int32_t)*attr->rangeLower) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
 
                if (attr->rangeUpper) {
                        if ((int32_t)t > (int32_t)*attr->rangeLower) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
 
@@ -796,13 +796,13 @@ static WERROR dsdb_syntax_NTTIME_validate_ldb(const struct dsdb_syntax_ctx *ctx,
 
                if (attr->rangeLower) {
                        if ((int32_t)t < (int32_t)*attr->rangeLower) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
 
                if (attr->rangeUpper) {
                        if ((int32_t)t > (int32_t)*attr->rangeLower) {
-                               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                               return WERR_DS_CONSTRAINT_VIOLATION;
                        }
                }
        }
@@ -887,13 +887,13 @@ static WERROR dsdb_syntax_DATA_BLOB_validate_one_val(const struct dsdb_syntax_ct
 
        if (attr->rangeLower) {
                if ((uint32_t)val->length < (uint32_t)*attr->rangeLower) {
-                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                       return WERR_DS_CONSTRAINT_VIOLATION;
                }
        }
 
        if (attr->rangeUpper) {
                if ((uint32_t)val->length > (uint32_t)*attr->rangeUpper) {
-                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                       return WERR_DS_CONSTRAINT_VIOLATION;
                }
        }
 
@@ -1598,13 +1598,13 @@ static WERROR dsdb_syntax_UNICODE_validate_one_val(const struct dsdb_syntax_ctx
 
        if (attr->rangeLower) {
                if ((size/2) < *attr->rangeLower) {
-                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                       return WERR_DS_CONSTRAINT_VIOLATION;
                }
        }
 
        if (attr->rangeUpper) {
                if ((size/2) > *attr->rangeUpper) {
-                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                       return WERR_DS_CONSTRAINT_VIOLATION;
                }
        }