Don't steal when we know the ptr will be null. Thanks to Simo for
authorJeremy Allison <jra@samba.org>
Fri, 22 May 2009 01:48:17 +0000 (18:48 -0700)
committerJeremy Allison <jra@samba.org>
Fri, 22 May 2009 01:49:24 +0000 (18:49 -0700)
pointing this out.
Jeremy.

source3/groupdb/mapping_ldb.c

index 2bb25b2e41087b196c3d033c91fe6c9389d06e25..97f84f1a803320cb275fb2bf4362c734acf316ea 100644 (file)
@@ -510,12 +510,11 @@ static NTSTATUS enum_aliasmem(const DOM_SID *alias, DOM_SID **sids, size_t *num)
 
        ret = ldb_search(ldb, ldb, &res, dn, LDB_SCOPE_BASE, attrs, NULL);
        if (ret == LDB_SUCCESS && res->count == 0) {
-               talloc_steal(dn, res);
+               talloc_free(res);
                talloc_free(dn);
                return NT_STATUS_OK;
        }
        if (ret != LDB_SUCCESS) {
-               talloc_steal(dn, res);
                talloc_free(dn);
                return NT_STATUS_INTERNAL_DB_CORRUPTION;
        }