summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2011-09-07 13:15:49 +0200
committerStephen Gallagher <sgallagh@redhat.com>2011-09-07 15:56:25 -0400
commita2a954c4186aaa9e9dd027aebb986062fc5670e7 (patch)
treef6afa626f64404ed5d85c2f1c072e538ba3031f8
parent14765d35f9440e3ca4fe771f162daf5c066ddd87 (diff)
downloadsssd_unused-a2a954c4186aaa9e9dd027aebb986062fc5670e7.tar.gz
sssd_unused-a2a954c4186aaa9e9dd027aebb986062fc5670e7.tar.xz
sssd_unused-a2a954c4186aaa9e9dd027aebb986062fc5670e7.zip
Do not access memory out of bounds
-rw-r--r--src/providers/ipa/ipa_hbac_hosts.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/providers/ipa/ipa_hbac_hosts.c b/src/providers/ipa/ipa_hbac_hosts.c
index dd82f284..4aea8608 100644
--- a/src/providers/ipa/ipa_hbac_hosts.c
+++ b/src/providers/ipa/ipa_hbac_hosts.c
@@ -499,10 +499,10 @@ hbac_shost_attrs_to_rule(TALLOC_CTX *mem_ctx,
goto done;
}
- for (idx = host_count; idx <= host_count + el->num_values; idx++) {
+ for (idx = host_count; idx < host_count + el->num_values; idx++) {
shosts->names[idx] =
talloc_strdup(shosts->names,
- (const char *)el->values[idx].data);
+ (const char *)el->values[idx - host_count].data);
if (shosts->names[idx] == NULL) {
ret = ENOMEM;
goto done;