Re: [SSSD] IFP: use default limit if provided is 0

[Petr Cech]

On 08/13/2015 12:48 PM, Pavel Březina wrote:

 From eef083f774988fe8e6b6a5a8513a163fd7558b55 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pavel=20B=C5=99ezina?=pbrez...@redhat.com
Date: Thu, 13 Aug 2015 12:46:59 +0200
Subject: [PATCH] IFP: use default limit if provided is 0

Hi,

CI: http://sssd-ci.duckdns.org/logs/job/21/49/summary.html
I compiled it, ran it and it worked.

ACK

Petr
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Re: [SSSD] IFP: use default limit if provided is 0

[Jakub Hrozek]

On Fri, Aug 14, 2015 at 01:07:34PM +0200, Petr Cech wrote:
 On 08/13/2015 12:48 PM, Pavel Březina wrote:
  From eef083f774988fe8e6b6a5a8513a163fd7558b55 Mon Sep 17 00:00:00 2001
 From: =?UTF-8?q?Pavel=20B=C5=99ezina?=pbrez...@redhat.com
 Date: Thu, 13 Aug 2015 12:46:59 +0200
 Subject: [PATCH] IFP: use default limit if provided is 0
 Hi,
 
 CI: http://sssd-ci.duckdns.org/logs/job/21/49/summary.html
 I compiled it, ran it and it worked.
 
 ACK
 
 Petr

* master: ef7de95fc4827a660254a942fa394f34ed9694a9
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

[SSSD] IFP: use default limit if provided is 0

[Pavel Březina]

 From eef083f774988fe8e6b6a5a8513a163fd7558b55 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrez...@redhat.com
Date: Thu, 13 Aug 2015 12:46:59 +0200
Subject: [PATCH] IFP: use default limit if provided is 0

Returning zero values doesn't make any sense, so we may use it as
use sssd configuration instead.
---
 src/responder/ifp/ifpsrv_util.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/responder/ifp/ifpsrv_util.c b/src/responder/ifp/ifpsrv_util.c
index 3b02fd06f5227e4ffc3d40ffb20fed981c5028a7..904c4f62ec5653a534877fd6870832128720b694 100644
--- a/src/responder/ifp/ifpsrv_util.c
+++ b/src/responder/ifp/ifpsrv_util.c
@@ -274,7 +274,9 @@ ifp_is_user_attr_allowed(struct ifp_ctx *ifp_ctx, const char *attr)
 
 static uint32_t ifp_list_limit(struct ifp_ctx *ctx, uint32_t limit)
 {
-if (ctx-wildcard_limit) {
+if (limit == 0) {
+return ctx-wildcard_limit;
+} else if (ctx-wildcard_limit) {
 return MIN(ctx-wildcard_limit, limit);
 } else {
 return limit;
-- 
1.9.3

___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel