dsdb: Use attribute-name parameter for error message
authorTim Beale <timbeale@catalyst.net.nz>
Wed, 4 Apr 2018 22:40:03 +0000 (10:40 +1200)
committerGarming Sam <garming@samba.org>
Fri, 11 May 2018 04:01:24 +0000 (06:01 +0200)
We'll reuse this code for working out the msDS-ResultantPSO, so
references to 'tokenGroups' in error messages would be misleading.

Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Garming Sam <garming@catalyst.net.nz>
Signed-off-by: Tim Beale <timbeale@catalyst.net.nz>
source4/dsdb/samdb/ldb_modules/operational.c

index 576cf8615740e27ec6704b77db8f891bc0dae553..3fe61edcfd1cfa0afc6c97d5ca6a18f5eb48e074 100644 (file)
@@ -249,8 +249,9 @@ static int construct_generic_token_groups(struct ldb_module *module,
                                           tmp_ctx, &groupSIDs, &num_groupSIDs);
 
        if (!NT_STATUS_IS_OK(status)) {
-               ldb_asprintf_errstring(ldb, "Failed to construct tokenGroups: expanding groups of SID %s failed: %s",
-                                      account_sid_string, nt_errstr(status));
+               ldb_asprintf_errstring(ldb, "Failed to construct %s: expanding groups of SID %s failed: %s",
+                                      attribute_string, account_sid_string,
+                                      nt_errstr(status));
                talloc_free(tmp_ctx);
                return LDB_ERR_OPERATIONS_ERROR;
        }
@@ -264,8 +265,9 @@ static int construct_generic_token_groups(struct ldb_module *module,
        status = dsdb_expand_nested_groups(ldb, &primary_group_blob, false, filter,
                                           tmp_ctx, &groupSIDs, &num_groupSIDs);
        if (!NT_STATUS_IS_OK(status)) {
-               ldb_asprintf_errstring(ldb, "Failed to construct tokenGroups: expanding groups of SID %s failed: %s",
-                                      account_sid_string, nt_errstr(status));
+               ldb_asprintf_errstring(ldb, "Failed to construct %s: expanding groups of SID %s failed: %s",
+                                      attribute_string, account_sid_string,
+                                      nt_errstr(status));
                talloc_free(tmp_ctx);
                return LDB_ERR_OPERATIONS_ERROR;
        }