dsdb: Fix CID 1433614 Dereference after null check
authorVolker Lendecke <vl@samba.org>
Wed, 28 Mar 2018 12:10:59 +0000 (07:10 -0500)
committerJeremy Allison <jra@samba.org>
Wed, 28 Mar 2018 22:21:54 +0000 (00:21 +0200)
This whole routine assumes module!=NULL, both in the successful as
well as in error cases. So checking for module!=NULL is confusing both
the reader as well as Coverity.

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source4/dsdb/samdb/ldb_modules/partition.c

index 422ed369ff55ab7d2fea78d424534925f18c33bf..37e714d6e1be78e0c79695efdf28ffda84eb659b 100644 (file)
@@ -1016,7 +1016,8 @@ int partition_del_trans(struct ldb_module *module)
                                                              struct partition_private_data);
 
        for (i=0; data && data->partitions && data->partitions[i]; i++) {
-               if ((module && ldb_module_flags(ldb_module_get_ctx(module)) & LDB_FLG_ENABLE_TRACING)) {
+               if (ldb_module_flags(ldb_module_get_ctx(module)) &
+                   LDB_FLG_ENABLE_TRACING) {
                        ldb_debug(ldb_module_get_ctx(module), LDB_DEBUG_TRACE, "partition_del_trans() -> %s",
                                  ldb_dn_get_linearized(data->partitions[i]->ctrl->dn));
                }
@@ -1035,7 +1036,8 @@ int partition_del_trans(struct ldb_module *module)
        }
        data->in_transaction--;
 
-       if ((module && ldb_module_flags(ldb_module_get_ctx(module)) & LDB_FLG_ENABLE_TRACING)) {
+       if (ldb_module_flags(ldb_module_get_ctx(module)) &
+           LDB_FLG_ENABLE_TRACING) {
                ldb_debug(ldb_module_get_ctx(module), LDB_DEBUG_TRACE, "partition_del_trans() -> (metadata partition)");
        }
        ret = ldb_next_del_trans(module);