summaryrefslogtreecommitdiff
path: root/source4/dsdb
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>2009-11-13 19:31:40 +1100
committerAndrew Tridgell <tridge@samba.org>2009-11-13 19:32:47 +1100
commitc8884e1cf2f9fe08b464c34c25ce682ff3450b34 (patch)
treee710ba233d8d2a7a0a874d675d9467394c09de47 /source4/dsdb
parentdbae29de1c146d1c6d2e1e0a2a82593f85c8ba0d (diff)
downloadsamba-c8884e1cf2f9fe08b464c34c25ce682ff3450b34.tar.gz
samba-c8884e1cf2f9fe08b464c34c25ce682ff3450b34.tar.bz2
samba-c8884e1cf2f9fe08b464c34c25ce682ff3450b34.zip
s4-ldb: changed the DN checks for \n to warnings
a \n is sometimes allowed in AD (eg in deleted DNs). Until we know when is really is allowed, treat it as a warning only.
Diffstat (limited to 'source4/dsdb')
-rw-r--r--source4/dsdb/common/tests/dsdb_dn.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/source4/dsdb/common/tests/dsdb_dn.c b/source4/dsdb/common/tests/dsdb_dn.c
index 8f1aeca082..183bce28d0 100644
--- a/source4/dsdb/common/tests/dsdb_dn.c
+++ b/source4/dsdb/common/tests/dsdb_dn.c
@@ -317,9 +317,12 @@ static bool torture_dsdb_dn_invalid(struct torture_context *torture)
/* Check DN based on MS-ADTS:3.1.1.5.1.2 Naming Constraints*/
val = data_blob_string_const("CN=New\nLine,DC=SAMBA,DC=org");
- torture_assert(torture,
- dsdb_dn_parse(mem_ctx, ldb, &val, LDB_SYNTAX_DN) == NULL,
- "Should have Failed to create a DN with 0xA in it");
+
+ /* changed to a warning until we understand the DEL: DNs */
+ if (dsdb_dn_parse(mem_ctx, ldb, &val, LDB_SYNTAX_DN) != NULL) {
+ torture_warning(torture,
+ "Should have Failed to create a DN with 0xA in it");
+ }
val = data_blob_string_const("B:4:ABAB:CN=New\nLine,DC=SAMBA,DC=org");
torture_assert(torture,