diff options
| author | Kamen Mazdrashki <kamenim@samba.org> | 2010-04-08 05:30:16 +0300 |
|---|---|---|
| committer | Kamen Mazdrashki <kamenim@samba.org> | 2010-04-09 12:21:31 +0300 |
| commit | c3d77989913a1ec270a50847922f6c1ffd7cbad0 (patch) | |
| tree | c232e221253eaf614862db166e7e4218810a4291 /source4/dsdb/schema/schema_prefixmap.c | |
| parent | e5ef11f4e29af77147dc21e6e8986ed091818566 (diff) | |
| download | samba-c3d77989913a1ec270a50847922f6c1ffd7cbad0.tar.gz samba-c3d77989913a1ec270a50847922f6c1ffd7cbad0.tar.bz2 samba-c3d77989913a1ec270a50847922f6c1ffd7cbad0.zip | |
s4/dsdb: Use dsdb_schema_info object to verify schema_info blobs
Diffstat (limited to 'source4/dsdb/schema/schema_prefixmap.c')
| -rw-r--r-- | source4/dsdb/schema/schema_prefixmap.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/source4/dsdb/schema/schema_prefixmap.c b/source4/dsdb/schema/schema_prefixmap.c index 0ed078fa32..50f74b7161 100644 --- a/source4/dsdb/schema/schema_prefixmap.c +++ b/source4/dsdb/schema/schema_prefixmap.c @@ -393,22 +393,23 @@ static WERROR _dsdb_drsuapi_pfm_verify(const struct drsuapi_DsReplicaOIDMapping_ num_mappings = ctr->num_mappings; if (have_schema_info) { + DATA_BLOB blob; + struct dsdb_schema_info *schi = NULL; + if (ctr->num_mappings < 2) { return WERR_INVALID_PARAMETER; } /* check last entry for being special */ mapping = &ctr->mappings[ctr->num_mappings - 1]; - if (!mapping->oid.binary_oid) { - return WERR_INVALID_PARAMETER; - } if (mapping->id_prefix != 0) { return WERR_INVALID_PARAMETER; } - if (mapping->oid.length != 21) { - return WERR_INVALID_PARAMETER; - } - if (*mapping->oid.binary_oid != 0xFF) { + + /* parse schemaInfo blob to verify it is valid */ + blob = data_blob_const(mapping->oid.binary_oid, mapping->oid.length); + if (!W_ERROR_IS_OK(dsdb_schema_info_from_blob(&blob, talloc_autofree_context(), &schi))) { + talloc_free(schi); return WERR_INVALID_PARAMETER; } |