debug it...
[metze/samba/wip.git] / source4 / dsdb / schema / schema_syntax.c
index 75dc16252b5c41ca512ce9a47fdc5f5552714e0a..e82b350d2004de5eb0030480b60ffd542ccefc6f 100644 (file)
@@ -97,7 +97,7 @@ static bool dsdb_syntax_attid_from_remote_attid(const struct dsdb_syntax_ctx *ct
                return false;
        }
 
-       werr = dsdb_schema_pfm_make_attid(ctx->schema->prefixmap, oid, id_local);
+       werr = dsdb_schema_pfm_attid_from_oid(ctx->schema->prefixmap, oid, id_local);
        if (!W_ERROR_IS_OK(werr)) {
                DEBUG(0,("OID->ATTID failed (%s) for: %s\n", win_errstr(werr), oid));
                return false;
@@ -209,7 +209,7 @@ static WERROR dsdb_syntax_BOOL_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ctx,
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -229,9 +229,11 @@ static WERROR dsdb_syntax_BOOL_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ctx,
                blobs[i] = data_blob_talloc(blobs, NULL, 4);
                W_ERROR_HAVE_NO_MEMORY(blobs[i].data);
 
-               if (strcmp("TRUE", (const char *)in->values[i].data) == 0) {
+               if (in->values[i].length >= 4 &&
+                   strncmp("TRUE", (const char *)in->values[i].data, in->values[i].length) == 0) {
                        SIVAL(blobs[i].data, 0, 0x00000001);
-               } else if (strcmp("FALSE", (const char *)in->values[i].data) == 0) {
+               } else if (in->values[i].length >= 5 &&
+                          strncmp("FALSE", (const char *)in->values[i].data, in->values[i].length) == 0) {
                        SIVAL(blobs[i].data, 0, 0x00000000);
                } else {
                        return WERR_FOOBAR;
@@ -248,26 +250,27 @@ static WERROR dsdb_syntax_BOOL_validate_ldb(const struct dsdb_syntax_ctx *ctx,
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
-               int t, f;
-
                if (in->values[i].length == 0) {
                        return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
                }
 
-               t = strncmp("TRUE",
+               if (in->values[i].length >= 4 &&
+                   strncmp("TRUE",
                            (const char *)in->values[i].data,
-                           in->values[i].length);
-               f = strncmp("FALSE",
+                           in->values[i].length) == 0) {
+                       continue;
+               }
+               if (in->values[i].length >= 5 &&
+                   strncmp("FALSE",
                            (const char *)in->values[i].data,
-                           in->values[i].length);
-
-               if (t != 0 && f != 0) {
-                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+                           in->values[i].length) == 0) {
+                       continue;
                }
+               return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
        }
 
        return WERR_OK;
@@ -322,7 +325,7 @@ static WERROR dsdb_syntax_INT32_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ctx
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -361,11 +364,12 @@ static WERROR dsdb_syntax_INT32_validate_ldb(const struct dsdb_syntax_ctx *ctx,
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
                long v;
+               long n;
                char buf[sizeof("-2147483648")];
                char *end = NULL;
 
@@ -384,6 +388,28 @@ static WERROR dsdb_syntax_INT32_validate_ldb(const struct dsdb_syntax_ctx *ctx,
                        return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
                }
 
+               n = v & UINT32_MAX;
+
+               if (n != v) {
+                       DEBUG(0,("invalid[%s] n[%ld] v[%ld]\n",
+                               buf, n, v));
+                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+               }
+
+               if ((v & INT32_MIN) && buf[0] != '-') {
+                       /*
+                        * if the 0x80000000 bit is set, it is a negative
+                        * value. We need to make sure the it was given
+                        * as a negativ string value.
+                        *
+                        * We need to accept '-2147483647', but reject
+                        * '2147483649', both represent 0x80000001.
+                        */
+                       DEBUG(0,("invalid[%s] n[%ld] v[%ld]\n",
+                               buf, n, v));
+                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+               }
+
                if (attr->rangeLower) {
                        if ((int32_t)v < (int32_t)*attr->rangeLower) {
                                return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
@@ -449,7 +475,7 @@ static WERROR dsdb_syntax_INT64_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ctx
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -486,11 +512,12 @@ static WERROR dsdb_syntax_INT64_validate_ldb(const struct dsdb_syntax_ctx *ctx,
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
                long long v;
+               long long n;
                char buf[sizeof("-9223372036854775808")];
                char *end = NULL;
 
@@ -509,6 +536,30 @@ static WERROR dsdb_syntax_INT64_validate_ldb(const struct dsdb_syntax_ctx *ctx,
                        return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
                }
 
+               n = v & UINT64_MAX;
+
+               if (n != v) {
+                       DEBUG(0,("invalid[%s] n[%lld] v[%lld]\n",
+                               buf, n, v));
+                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+               }
+
+               if ((v & INT64_MIN) && buf[0] != '-') {
+                       /*
+                        * if the 0x8000000000000000 bit is set,
+                        * it is a negative value. We need to
+                        * make sure the it was given as a negative
+                        * string value.
+                        *
+                        * We need to accept '-9223372036854775807',
+                        * but reject '9223372036854775809',
+                        * both represent 0x8000000000000001.
+                        */
+                       DEBUG(0,("invalid[%s] n[%lld] v[%lld]\n",
+                               buf, n, v));
+                       return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
+               }
+
                if (attr->rangeLower) {
                        if ((int64_t)v < (int64_t)*attr->rangeLower) {
                                return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
@@ -554,6 +605,11 @@ static WERROR dsdb_syntax_NTTIME_UTC_drsuapi_to_ldb(const struct dsdb_syntax_ctx
                }
 
                v = BVAL(in->value_ctr.values[i].blob->data, 0);
+               if (v == 0) {
+                       /* special case for 1601 zero timestamp */
+                       out->values[i] = data_blob_string_const("16010101000000.0Z");
+                       continue;
+               }
                v *= 10000000;
                t = nt_time_to_unix(v);
 
@@ -584,7 +640,7 @@ static WERROR dsdb_syntax_NTTIME_UTC_ldb_to_drsuapi(const struct dsdb_syntax_ctx
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -607,6 +663,11 @@ static WERROR dsdb_syntax_NTTIME_UTC_ldb_to_drsuapi(const struct dsdb_syntax_ctx
                blobs[i] = data_blob_talloc(blobs, NULL, 8);
                W_ERROR_HAVE_NO_MEMORY(blobs[i].data);
 
+               if (ldb_val_string_cmp(&in->values[i], "16010101000000.0Z") == 0) {
+                       SBVALS(blobs[i].data, 0, 0);
+                       continue;
+               }
+
                t = ldb_string_utc_to_time((const char *)in->values[i].data);
                unix_to_nt_time(&v, t);
                v /= 10000000;
@@ -624,7 +685,7 @@ static WERROR dsdb_syntax_NTTIME_UTC_validate_ldb(const struct dsdb_syntax_ctx *
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -649,7 +710,7 @@ static WERROR dsdb_syntax_NTTIME_UTC_validate_ldb(const struct dsdb_syntax_ctx *
                }
 
                if (attr->rangeUpper) {
-                       if ((int32_t)t > (int32_t)*attr->rangeLower) {
+                       if ((int32_t)t > (int32_t)*attr->rangeUpper) {
                                return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
                        }
                }
@@ -693,6 +754,11 @@ static WERROR dsdb_syntax_NTTIME_drsuapi_to_ldb(const struct dsdb_syntax_ctx *ct
                }
 
                v = BVAL(in->value_ctr.values[i].blob->data, 0);
+               if (v == 0) {
+                       /* special case for 1601 zero timestamp */
+                       out->values[i] = data_blob_string_const("16010101000000.0Z");
+                       continue;
+               }
                v *= 10000000;
                t = nt_time_to_unix(v);
 
@@ -715,7 +781,7 @@ static WERROR dsdb_syntax_NTTIME_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ct
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -739,6 +805,11 @@ static WERROR dsdb_syntax_NTTIME_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ct
                blobs[i] = data_blob_talloc(blobs, NULL, 8);
                W_ERROR_HAVE_NO_MEMORY(blobs[i].data);
 
+               if (ldb_val_string_cmp(&in->values[i], "16010101000000.0Z") == 0) {
+                       SBVALS(blobs[i].data, 0, 0);
+                       continue;
+               }
+
                ret = ldb_val_to_time(&in->values[i], &t);
                if (ret != LDB_SUCCESS) {
                        return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
@@ -759,7 +830,7 @@ static WERROR dsdb_syntax_NTTIME_validate_ldb(const struct dsdb_syntax_ctx *ctx,
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -778,7 +849,7 @@ static WERROR dsdb_syntax_NTTIME_validate_ldb(const struct dsdb_syntax_ctx *ctx,
                }
 
                if (attr->rangeUpper) {
-                       if ((int32_t)t > (int32_t)*attr->rangeLower) {
+                       if ((int32_t)t > (int32_t)*attr->rangeUpper) {
                                return WERR_DS_INVALID_ATTRIBUTE_SYNTAX;
                        }
                }
@@ -813,7 +884,7 @@ static WERROR dsdb_syntax_DATA_BLOB_drsuapi_to_ldb(const struct dsdb_syntax_ctx
                }
 
                out->values[i] = data_blob_dup_talloc(out->values,
-                                                     in->value_ctr.values[i].blob);
+                                                     *in->value_ctr.values[i].blob);
                W_ERROR_HAVE_NO_MEMORY(out->values[i].data);
        }
 
@@ -830,7 +901,7 @@ static WERROR dsdb_syntax_DATA_BLOB_ldb_to_drsuapi(const struct dsdb_syntax_ctx
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -847,7 +918,7 @@ static WERROR dsdb_syntax_DATA_BLOB_ldb_to_drsuapi(const struct dsdb_syntax_ctx
        for (i=0; i < in->num_values; i++) {
                out->value_ctr.values[i].blob   = &blobs[i];
 
-               blobs[i] = data_blob_dup_talloc(blobs, &in->values[i]);
+               blobs[i] = data_blob_dup_talloc(blobs, in->values[i]);
                W_ERROR_HAVE_NO_MEMORY(blobs[i].data);
        }
 
@@ -859,7 +930,7 @@ static WERROR dsdb_syntax_DATA_BLOB_validate_one_val(const struct dsdb_syntax_ct
                                                     const struct ldb_val *val)
 {
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        if (attr->rangeLower) {
@@ -885,7 +956,7 @@ static WERROR dsdb_syntax_DATA_BLOB_validate_ldb(const struct dsdb_syntax_ctx *c
        WERROR status;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -973,7 +1044,7 @@ static WERROR _dsdb_syntax_OID_obj_drsuapi_to_ldb(const struct dsdb_syntax_ctx *
        W_ERROR_HAVE_NO_MEMORY(out->values);
 
        for (i=0; i < out->num_values; i++) {
-               uint32_t v;
+               uint32_t v, vo;
                const struct dsdb_class *c;
                const char *str;
 
@@ -986,6 +1057,7 @@ static WERROR _dsdb_syntax_OID_obj_drsuapi_to_ldb(const struct dsdb_syntax_ctx *
                }
 
                v = IVAL(in->value_ctr.values[i].blob->data, 0);
+               vo = v;
 
                /* convert remote ATTID to local ATTID */
                if (!dsdb_syntax_attid_from_remote_attid(ctx, mem_ctx, v, &v)) {
@@ -995,8 +1067,11 @@ static WERROR _dsdb_syntax_OID_obj_drsuapi_to_ldb(const struct dsdb_syntax_ctx *
 
                c = dsdb_class_by_governsID_id(ctx->schema, v);
                if (!c) {
-                       DEBUG(1,(__location__ ": Unknown governsID 0x%08X\n", v));
-                       return WERR_FOOBAR;
+                       int dbg_level = ctx->schema->resolving_in_progress ? 10 : 0;
+                       DEBUG(dbg_level,(__location__ ": %s unknown local governsID 0x%08X remote 0x%08X%s\n",
+                             attr->lDAPDisplayName, v, vo,
+                             ctx->schema->resolving_in_progress ? "resolving in progress" : ""));
+                       return WERR_DS_OBJ_CLASS_NOT_DEFINED;
                }
 
                str = talloc_strdup(out->values, c->lDAPDisplayName);
@@ -1026,19 +1101,22 @@ static WERROR _dsdb_syntax_OID_attr_drsuapi_to_ldb(const struct dsdb_syntax_ctx
        W_ERROR_HAVE_NO_MEMORY(out->values);
 
        for (i=0; i < out->num_values; i++) {
-               uint32_t v;
+               uint32_t v, vo;
                const struct dsdb_attribute *a;
                const char *str;
 
                if (in->value_ctr.values[i].blob == NULL) {
+                       DEBUG(0, ("Attribute has no value\n"));
                        return WERR_FOOBAR;
                }
 
                if (in->value_ctr.values[i].blob->length != 4) {
+                       DEBUG(0, ("Attribute has a value with 0 length\n"));
                        return WERR_FOOBAR;
                }
 
                v = IVAL(in->value_ctr.values[i].blob->data, 0);
+               vo = v;
 
                /* convert remote ATTID to local ATTID */
                if (!dsdb_syntax_attid_from_remote_attid(ctx, mem_ctx, v, &v)) {
@@ -1048,8 +1126,11 @@ static WERROR _dsdb_syntax_OID_attr_drsuapi_to_ldb(const struct dsdb_syntax_ctx
 
                a = dsdb_attribute_by_attributeID_id(ctx->schema, v);
                if (!a) {
-                       DEBUG(1,(__location__ ": Unknown attributeID_id 0x%08X\n", v));
-                       return WERR_FOOBAR;
+                       int dbg_level = ctx->schema->resolving_in_progress ? 10 : 0;
+                       DEBUG(dbg_level,(__location__ ": %s unknown local attributeID_id 0x%08X remote 0x%08X%s\n",
+                             attr->lDAPDisplayName, v, vo,
+                             ctx->schema->resolving_in_progress ? "resolving in progress" : ""));
+                       return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
                }
 
                str = talloc_strdup(out->values, a->lDAPDisplayName);
@@ -1241,6 +1322,7 @@ static WERROR _dsdb_syntax_OID_attr_ldb_to_drsuapi(const struct dsdb_syntax_ctx
 
                obj_attr = dsdb_attribute_by_lDAPDisplayName(ctx->schema, (const char *)in->values[i].data);
                if (!obj_attr) {
+                       DEBUG(0, ("Unable to find attribute %s in the schema\n", (const char *)in->values[i].data));
                        return WERR_FOOBAR;
                }
                SIVAL(blobs[i].data, 0, obj_attr->attributeID_id);
@@ -1343,7 +1425,7 @@ static WERROR dsdb_syntax_OID_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ctx,
                                             struct drsuapi_DsReplicaAttribute *out)
 {
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        switch (attr->attributeID_id) {
@@ -1426,7 +1508,7 @@ static WERROR dsdb_syntax_OID_validate_ldb(const struct dsdb_syntax_ctx *ctx,
        TALLOC_CTX *tmp_ctx;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        switch (attr->attributeID_id) {
@@ -1503,7 +1585,7 @@ static WERROR dsdb_syntax_UNICODE_drsuapi_to_ldb(const struct dsdb_syntax_ctx *c
                        return WERR_FOOBAR;
                }
 
-               out->values[i] = data_blob_string_const(str);
+               out->values[i] = data_blob_const(str, converted_size);
        }
 
        return WERR_OK;
@@ -1519,7 +1601,7 @@ static WERROR dsdb_syntax_UNICODE_ldb_to_drsuapi(const struct dsdb_syntax_ctx *c
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -1556,7 +1638,7 @@ static WERROR dsdb_syntax_UNICODE_validate_one_val(const struct dsdb_syntax_ctx
        bool ok;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        ok = convert_string_talloc(ctx->ldb,
@@ -1593,7 +1675,7 @@ static WERROR dsdb_syntax_UNICODE_validate_ldb(const struct dsdb_syntax_ctx *ctx
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -1732,7 +1814,7 @@ static WERROR dsdb_syntax_DN_ldb_to_drsuapi(const struct dsdb_syntax_ctx *ctx,
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -1814,7 +1896,7 @@ static WERROR dsdb_syntax_DN_validate_one_val(const struct dsdb_syntax_ctx *ctx,
        W_ERROR_HAVE_NO_MEMORY(tmp_ctx);
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        dsdb_dn = dsdb_dn_parse(tmp_ctx, ctx->ldb, val,
@@ -1828,7 +1910,7 @@ static WERROR dsdb_syntax_DN_validate_one_val(const struct dsdb_syntax_ctx *ctx,
        dn2 = ldb_dn_copy(tmp_ctx, dn);
        if (dn == NULL) {
                talloc_free(tmp_ctx);
-               return WERR_NOMEM;
+               return WERR_NOT_ENOUGH_MEMORY;
        }
 
        num_components = ldb_dn_get_comp_num(dn);
@@ -1897,7 +1979,7 @@ static WERROR dsdb_syntax_DN_validate_ldb(const struct dsdb_syntax_ctx *ctx,
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -1983,20 +2065,21 @@ static WERROR dsdb_syntax_DN_BINARY_drsuapi_to_ldb(const struct dsdb_syntax_ctx
                        W_ERROR_HAVE_NO_MEMORY(dn);
                }
 
-               status = GUID_to_ndr_blob(&id3.guid, tmp_ctx, &guid_blob);
-               if (!NT_STATUS_IS_OK(status)) {
-                       talloc_free(tmp_ctx);
-                       return ntstatus_to_werror(status);
-               }
+               if (!GUID_all_zero(&id3.guid)) {
+                       status = GUID_to_ndr_blob(&id3.guid, tmp_ctx, &guid_blob);
+                       if (!NT_STATUS_IS_OK(status)) {
+                               talloc_free(tmp_ctx);
+                               return ntstatus_to_werror(status);
+                       }
 
-               ret = ldb_dn_set_extended_component(dn, "GUID", &guid_blob);
-               if (ret != LDB_SUCCESS) {
-                       talloc_free(tmp_ctx);
-                       return WERR_FOOBAR;
+                       ret = ldb_dn_set_extended_component(dn, "GUID", &guid_blob);
+                       if (ret != LDB_SUCCESS) {
+                               talloc_free(tmp_ctx);
+                               return WERR_FOOBAR;
+                       }
+                       talloc_free(guid_blob.data);
                }
 
-               talloc_free(guid_blob.data);
-
                if (id3.__ndr_size_sid) {
                        DATA_BLOB sid_blob;
                        ndr_err = ndr_push_struct_blob(&sid_blob, tmp_ctx, &id3.sid,
@@ -2038,7 +2121,7 @@ static WERROR dsdb_syntax_DN_BINARY_ldb_to_drsuapi(const struct dsdb_syntax_ctx
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -2117,7 +2200,7 @@ static WERROR dsdb_syntax_DN_BINARY_validate_ldb(const struct dsdb_syntax_ctx *c
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -2187,7 +2270,7 @@ static WERROR dsdb_syntax_DN_STRING_validate_ldb(const struct dsdb_syntax_ctx *c
        unsigned int i;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        for (i=0; i < in->num_values; i++) {
@@ -2282,7 +2365,7 @@ static WERROR dsdb_syntax_PRESENTATION_ADDRESS_ldb_to_drsuapi(const struct dsdb_
        DATA_BLOB *blobs;
 
        if (attr->attributeID_id == DRSUAPI_ATTID_INVALID) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        out->attid                      = dsdb_attribute_get_attid(attr,
@@ -2344,7 +2427,8 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .ldb_to_drsuapi         = dsdb_syntax_BOOL_ldb_to_drsuapi,
                .validate_ldb           = dsdb_syntax_BOOL_validate_ldb,
                .equality               = "booleanMatch",
-               .comment                = "Boolean"
+               .comment                = "Boolean",
+               .auto_normalise         = true
        },{
                .name                   = "Integer",
                .ldap_oid               = LDB_SYNTAX_INTEGER,
@@ -2355,7 +2439,8 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .validate_ldb           = dsdb_syntax_INT32_validate_ldb,
                .equality               = "integerMatch",
                .comment                = "Integer",
-               .ldb_syntax             = LDB_SYNTAX_SAMBA_INT32
+               .ldb_syntax             = LDB_SYNTAX_SAMBA_INT32,
+               .auto_normalise         = true
        },{
                .name                   = "String(Octet)",
                .ldap_oid               = LDB_SYNTAX_OCTET_STRING,
@@ -2366,6 +2451,8 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .validate_ldb           = dsdb_syntax_DATA_BLOB_validate_ldb,
                .equality               = "octetStringMatch",
                .comment                = "Octet String",
+               .userParameters         = true,
+               .ldb_syntax             = LDB_SYNTAX_SAMBA_OCTET_STRING
        },{
                .name                   = "String(Sid)",
                .ldap_oid               = LDB_SYNTAX_OCTET_STRING,
@@ -2396,7 +2483,8 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .drsuapi_to_ldb         = dsdb_syntax_INT32_drsuapi_to_ldb,
                .ldb_to_drsuapi         = dsdb_syntax_INT32_ldb_to_drsuapi,
                .validate_ldb           = dsdb_syntax_INT32_validate_ldb,
-               .ldb_syntax             = LDB_SYNTAX_SAMBA_INT32
+               .ldb_syntax             = LDB_SYNTAX_SAMBA_INT32,
+               .auto_normalise         = true
        },{
        /* not used in w2k3 forest */
                .name                   = "String(Numeric)",
@@ -2418,7 +2506,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .drsuapi_to_ldb         = dsdb_syntax_DATA_BLOB_drsuapi_to_ldb,
                .ldb_to_drsuapi         = dsdb_syntax_DATA_BLOB_ldb_to_drsuapi,
                .validate_ldb           = dsdb_syntax_DATA_BLOB_validate_ldb,
-               .ldb_syntax             = LDB_SYNTAX_OCTET_STRING,
+               .ldb_syntax             = LDB_SYNTAX_SAMBA_OCTET_STRING,
        },{
                .name                   = "String(Teletex)",
                .ldap_oid               = "1.2.840.113556.1.4.905",
@@ -2441,7 +2529,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .validate_ldb           = dsdb_syntax_DATA_BLOB_validate_ldb,
                .equality               = "caseExactIA5Match",
                .comment                = "Printable String",
-               .ldb_syntax             = LDB_SYNTAX_OCTET_STRING,
+               .ldb_syntax             = LDB_SYNTAX_SAMBA_OCTET_STRING,
        },{
                .name                   = "String(UTC-Time)",
                .ldap_oid               = "1.3.6.1.4.1.1466.115.121.1.53",
@@ -2452,6 +2540,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .validate_ldb           = dsdb_syntax_NTTIME_UTC_validate_ldb,
                .equality               = "generalizedTimeMatch",
                .comment                = "UTC Time",
+               .auto_normalise         = true
        },{
                .name                   = "String(Generalized-Time)",
                .ldap_oid               = "1.3.6.1.4.1.1466.115.121.1.24",
@@ -2462,7 +2551,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .validate_ldb           = dsdb_syntax_NTTIME_validate_ldb,
                .equality               = "generalizedTimeMatch",
                .comment                = "Generalized Time",
-               .ldb_syntax             = LDB_SYNTAX_UTC_TIME,
+               .auto_normalise         = true
        },{
        /* not used in w2k3 schema */
                .name                   = "String(Case Sensitive)",
@@ -2478,7 +2567,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                 * as Directory String (LDB_SYNTAX_DIRECTORY_STRING), but case sensitive.
                 * But according to ms docs binary compare should do the job:
                 * http://msdn.microsoft.com/en-us/library/cc223200(v=PROT.10).aspx */
-               .ldb_syntax             = LDB_SYNTAX_OCTET_STRING,
+               .ldb_syntax             = LDB_SYNTAX_SAMBA_OCTET_STRING,
        },{
                .name                   = "String(Unicode)",
                .ldap_oid               = LDB_SYNTAX_DIRECTORY_STRING,
@@ -2501,6 +2590,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
                .equality               = "integerMatch",
                .comment                = "Large Integer",
                .ldb_syntax             = LDB_SYNTAX_INTEGER,
+               .auto_normalise         = true
        },{
                .name                   = "String(NT-Sec-Desc)",
                .ldap_oid               = LDB_SYNTAX_SAMBA_SECURITY_DESCRIPTOR,
@@ -2572,7 +2662,7 @@ static const struct dsdb_syntax dsdb_syntaxes[] = {
        },{
        /* not used in w2k3 schema */
                .name                   = "Object(Access-Point)",
-               .ldap_oid               = "1.3.6.1.4.1.1466.115.121.1.2",
+               .ldap_oid               = DSDB_SYNTAX_ACCESS_POINT,
                .oMSyntax               = 127,
                .oMObjectClass          = OMOBJECTCLASS("\x2b\x0c\x02\x87\x73\x1c\x00\x85\x3e"),
                .attributeSyntax_oid    = "2.5.5.14",
@@ -2633,6 +2723,16 @@ const struct dsdb_syntax *dsdb_syntax_for_attribute(const struct dsdb_attribute
        unsigned int i;
 
        for (i=0; i < ARRAY_SIZE(dsdb_syntaxes); i++) {
+               /*
+                * We must pretend that userParameters was declared
+                * binary string, so we can store the 'UTF16' (not
+                * really string) structure as given over SAMR to samba
+                */
+               if (dsdb_syntaxes[i].userParameters &&
+                   (strcasecmp(attr->lDAPDisplayName, "userParameters") == 0))
+               {
+                       return &dsdb_syntaxes[i];
+               }
                if (attr->oMSyntax != dsdb_syntaxes[i].oMSyntax) continue;
 
                if (attr->oMObjectClass.length != dsdb_syntaxes[i].oMObjectClass.length) continue;
@@ -2653,45 +2753,94 @@ const struct dsdb_syntax *dsdb_syntax_for_attribute(const struct dsdb_attribute
        return NULL;
 }
 
-WERROR dsdb_attribute_drsuapi_to_ldb(struct ldb_context *ldb,
-                                    const struct dsdb_schema *schema,
-                                    const struct dsdb_schema_prefixmap *pfm_remote,
-                                    const struct drsuapi_DsReplicaAttribute *in,
-                                    TALLOC_CTX *mem_ctx,
-                                    struct ldb_message_element *out)
+WERROR dsdb_attribute_drsuapi_remote_to_local(const struct dsdb_syntax_ctx *ctx,
+                                             enum drsuapi_DsAttributeId remote_attid_as_enum,
+                                             enum drsuapi_DsAttributeId *local_attid_as_enum,
+                                             const struct dsdb_attribute **_sa)
 {
+       TALLOC_CTX *frame = talloc_stackframe();
        const struct dsdb_attribute *sa;
-       struct dsdb_syntax_ctx syntax_ctx;
        uint32_t attid_local;
+       bool ok;
 
-       /* use default syntax conversion context */
-       dsdb_syntax_ctx_init(&syntax_ctx, ldb, schema);
-       syntax_ctx.pfm_remote = pfm_remote;
+       if (ctx->pfm_remote == NULL) {
+               smb_panic(__location__);
+       }
 
-       switch (dsdb_pfm_get_attid_type(in->attid)) {
+       switch (dsdb_pfm_get_attid_type(remote_attid_as_enum)) {
        case DSDB_ATTID_TYPE_PFM:
                /* map remote ATTID to local ATTID */
-               if (!dsdb_syntax_attid_from_remote_attid(&syntax_ctx, mem_ctx, in->attid, &attid_local)) {
+               ok = dsdb_syntax_attid_from_remote_attid(ctx, frame,
+                                                        remote_attid_as_enum,
+                                                        &attid_local);
+               if (!ok) {
                        DEBUG(0,(__location__ ": Can't find local ATTID for 0x%08X\n",
-                                in->attid));
-                       return WERR_FOOBAR;
+                                remote_attid_as_enum));
+                       TALLOC_FREE(frame);
+                       return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
                }
                break;
        case DSDB_ATTID_TYPE_INTID:
                /* use IntId value directly */
-               attid_local = in->attid;
+               attid_local = remote_attid_as_enum;
                break;
        default:
                /* we should never get here */
                DEBUG(0,(__location__ ": Invalid ATTID type passed for conversion - 0x%08X\n",
-                        in->attid));
+                        remote_attid_as_enum));
+               TALLOC_FREE(frame);
                return WERR_INVALID_PARAMETER;
        }
 
-       sa = dsdb_attribute_by_attributeID_id(schema, attid_local);
+       sa = dsdb_attribute_by_attributeID_id(ctx->schema, attid_local);
        if (!sa) {
-               DEBUG(1,(__location__ ": Unknown attributeID_id 0x%08X\n", in->attid));
-               return WERR_FOOBAR;
+               int dbg_level = ctx->schema->resolving_in_progress ? 10 : 0;
+               DEBUG(dbg_level,(__location__ ": Unknown local attributeID_id 0x%08X remote 0x%08X%s\n",
+                     attid_local, remote_attid_as_enum,
+                     ctx->schema->resolving_in_progress ? "resolving in progress" : ""));
+               TALLOC_FREE(frame);
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
+       }
+
+       /*
+        * We return the same class of attid as we were given.  That
+        * is, we trust the remote server not to use an
+        * msDS-IntId value in the schema partition
+        */
+       if (local_attid_as_enum != NULL) {
+               *local_attid_as_enum = (enum drsuapi_DsAttributeId)attid_local;
+       }
+
+       if (_sa != NULL) {
+               *_sa = sa;
+       }
+
+       TALLOC_FREE(frame);
+       return WERR_OK;
+}
+
+WERROR dsdb_attribute_drsuapi_to_ldb(struct ldb_context *ldb,
+                                    const struct dsdb_schema *schema,
+                                    const struct dsdb_schema_prefixmap *pfm_remote,
+                                    const struct drsuapi_DsReplicaAttribute *in,
+                                    TALLOC_CTX *mem_ctx,
+                                    struct ldb_message_element *out,
+                                    enum drsuapi_DsAttributeId *local_attid_as_enum)
+{
+       struct dsdb_syntax_ctx syntax_ctx;
+       const struct dsdb_attribute *sa = NULL;
+       WERROR werr;
+
+       /* use default syntax conversion context */
+       dsdb_syntax_ctx_init(&syntax_ctx, ldb, schema);
+       syntax_ctx.pfm_remote = pfm_remote;
+
+       werr = dsdb_attribute_drsuapi_remote_to_local(&syntax_ctx,
+                                                     in->attid,
+                                                     local_attid_as_enum,
+                                                     &sa);
+       if (!W_ERROR_IS_OK(werr)) {
+               return werr;
        }
 
        return sa->syntax->drsuapi_to_ldb(&syntax_ctx, sa, in, mem_ctx, out);
@@ -2708,7 +2857,7 @@ WERROR dsdb_attribute_ldb_to_drsuapi(struct ldb_context *ldb,
 
        sa = dsdb_attribute_by_lDAPDisplayName(schema, in->name);
        if (!sa) {
-               return WERR_FOOBAR;
+               return WERR_DS_ATT_NOT_DEF_IN_SCHEMA;
        }
 
        /* use default syntax conversion context */