[PATCH] libipa_hbac cleanup patches
by Jakub Hrozek
Hi,
attached are patches that in my opinion make libipa_hbac easier to use
by external projects like pam_hbac.
[PATCH 1/4] libipa_hbac: Do not use C99
- even though we use C99 in the deamon, pam_hbac needs to run on very
exotic platforms. I don't think the code readability is decreased too
much
[PATCH 2/4] libipa_hbac: Add more debug messages
- just adds debug messages
[PATCH 3/4] libipa_hbac: Fix typo in constant name
- fixes a typo that breaks compilation on non-GNU systems
[PATCH 4/4] libipa_hbac: Move the library to src/lib/ipa_hbac
- This might be a bit controversial, but in my opinion a library
shouldn't be present in the provider subdirectory. Please also note
I added "I$(top_srcdir)/src/util", this is just so that I can copy the
libipa_hbac code without any changes to pam_hbac (even changes to
header file locations..)
8 years, 1 month
[PATCH] CLIENT: Retry request after EPIPE
by Lukas Slebodnik
ehlo,
It took me some time to reproduce issue with cron.
It occured very rarely in my case (twice in a week).
Therefore I prepared different reproducer "mini_cron.c"
attached in mail. It tries to query for data in the interval
(9.990 sec .. 10.069 sec) when responder might destroy connection.
mini_cron expect there is a user mof_user1 in LDAP and client_idle_timeout
has minimal value 10 seconds. Default is 60 seconds. It's also good to
decrease memory cache timeout to ensure connection to responder every time.
e.g.
[sssd]
config_file_version = 2
services = nss, pam
domains = LDAP
client_idle_timeout = 10
[nss]
filter_groups = root
filter_users = root
memcache_timeout = 0
client_idle_timeout = 10
debug_level =9
debug_microseconds = true
Detailed explanation is in commit message.
Attached is also a debug patch which I used as part of
analysis when it can fail.
BTW I ran mini_cron reprodurer for a week and it didn't fail.
LS
8 years, 1 month
[PATCH] build: detect endianness at configure time
by David Disseldorp
From: David Disseldorp <ddiss(a)samba.org>
WORDS_BIGENDIAN, HAVE_BIG_ENDIAN and HAVE_LITTLE_ENDIAN are needed by
Samba. See Samba's byteorder.h header for an example.
Signed-off-by: David Disseldorp <ddiss(a)samba.org>
---
configure.ac | 3 +++
1 file changed, 3 insertions(+)
diff --git a/configure.ac b/configure.ac
index 9f09927..9b674ba 100644
--- a/configure.ac
+++ b/configure.ac
@@ -76,6 +76,9 @@ AC_CHECK_FUNCS([ utimensat \
#Check for endian headers
AC_CHECK_HEADERS([endian.h sys/endian.h byteswap.h])
+AC_C_BIGENDIAN([AC_DEFINE(HAVE_BIG_ENDIAN, [1], [whether platform is big endian])],
+ [AC_DEFINE(HAVE_LITTLE_ENDIAN, [1], [whether platform is little endian])])
+
#Set the NSS library install path
AC_ARG_ENABLE([nsslibdir], [AS_HELP_STRING([--enable-nsslibdir],
[Where to install nss libraries ($libdir)])],
--
2.1.4
8 years, 1 month