diff options
| author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2011-03-04 10:44:22 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2011-03-04 22:07:24 +0100 |
| commit | 349b9b72ec36194a1275eaa42ca145071256b623 (patch) | |
| tree | 1727c1db47c347aae354f796bcbc3a13c00810c8 /source4/dsdb/samdb | |
| parent | 746194cfc4e0c119e1947f79ffc7cb50d50b9f43 (diff) | |
| download | samba-349b9b72ec36194a1275eaa42ca145071256b623.tar.gz samba-349b9b72ec36194a1275eaa42ca145071256b623.tar.bz2 samba-349b9b72ec36194a1275eaa42ca145071256b623.zip | |
s4:dsdb - we don't need to check if a DN != NULL if we call "ldb_dn_validate"
"ldb_dn_validate" is NULL-safe and does the check implicitly.
Reviewed by: Tridge
Diffstat (limited to 'source4/dsdb/samdb')
| -rw-r--r-- | source4/dsdb/samdb/ldb_modules/simple_ldap_map.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source4/dsdb/samdb/ldb_modules/simple_ldap_map.c b/source4/dsdb/samdb/ldb_modules/simple_ldap_map.c index fce4592895..7412d29267 100644 --- a/source4/dsdb/samdb/ldb_modules/simple_ldap_map.c +++ b/source4/dsdb/samdb/ldb_modules/simple_ldap_map.c @@ -147,7 +147,7 @@ static struct ldb_val objectCategory_always_dn(struct ldb_module *module, TALLOC const struct ldb_schema_attribute *a = ldb_schema_attribute_by_name(ldb, "objectCategory"); dn = ldb_dn_from_ldb_val(ctx, ldb, val); - if (dn && ldb_dn_validate(dn)) { + if (ldb_dn_validate(dn)) { talloc_free(dn); return val_copy(module, ctx, val); } |