diff options
| author | Andrew Tridgell <tridge@samba.org> | 2009-11-13 19:31:40 +1100 |
|---|---|---|
| committer | Andrew Tridgell <tridge@samba.org> | 2009-11-13 19:32:47 +1100 |
| commit | c8884e1cf2f9fe08b464c34c25ce682ff3450b34 (patch) | |
| tree | e710ba233d8d2a7a0a874d675d9467394c09de47 /source4/dsdb | |
| parent | dbae29de1c146d1c6d2e1e0a2a82593f85c8ba0d (diff) | |
| download | samba-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.c | 9 |
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, |