[SSSD] [PATCH] Fix a wrong return value in IPA HBAC

Sumit Bose sbose at redhat.com
Mon May 3 08:09:00 UTC 2010


Hi,

this patch corrects a return value in check_user() of the IPA access
provider.

bye,
Sumit
-------------- next part --------------
>From 428cec338b9e078178ec5ea8013a5f33af698d60 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Mon, 3 May 2010 10:06:39 +0200
Subject: [PATCH] Fix a wrong return value in IPA HBAC

---
 src/providers/ipa/ipa_access.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/providers/ipa/ipa_access.c b/src/providers/ipa/ipa_access.c
index 55a7133..3acf806 100644
--- a/src/providers/ipa/ipa_access.c
+++ b/src/providers/ipa/ipa_access.c
@@ -1368,7 +1368,7 @@ enum check_result check_user(struct hbac_ctx *hbac_ctx,
         return RULE_ERROR;
     }
     if (el->num_values == 0) {
-        DEBUG(9, ("USer category is not set.\n"));
+        DEBUG(9, ("User category is not set.\n"));
     } else {
         for (i = 0; i < el->num_values; i++) {
             if (strncasecmp("all", (const char *) el->values[i].data,
@@ -1389,7 +1389,7 @@ enum check_result check_user(struct hbac_ctx *hbac_ctx,
     }
     if (el->num_values == 0) {
         DEBUG(9, ("No user specified, rule does not apply.\n"));
-        return RULE_APPLICABLE;
+        return RULE_NOT_APPLICABLE;
     } else {
         for (i = 0; i < el->num_values; i++) {
             DEBUG(9, ("Searching matches for [%.*s].\n", el->values[i].length,
-- 
1.6.6.1



More information about the sssd-devel mailing list