sssd-devel October 2014

sssd-devel@lists.fedorahosted.org

16 participants
100 discussions

[PATCH] MONITOR: Remove useless memory contexts
by Jakub Hrozek 10 Oct '14

10 Oct '14

Hi, another small fix for the monitor code. I think the contexts come from way ago when we supported online configuration changes, but to me, they just make the code more confusing.

2 2

Re: [SSSD] [PATCH] Suppress warning maybe-uninitialized
by Lukas Slebodnik 09 Oct '14

09 Oct '14

On (08/10/14 00:16), Simo Sorce wrote: >On Tue, 7 Oct 2014 23:57:34 +0200 >Lukas Slebodnik <lslebodn(a)redhat.com> wrote: > >> - name = map[a].sys_name; >> + name = map[i].sys_name; >> + base_attr_idx = 1; > >this should be base_attr_idx = i; I think. > > /me facepalms fixed version attached. LS

3 2

[PATCH] UTIL: Do not depend on monitor code
by Jakub Hrozek 09 Oct '14

09 Oct '14

Hi, unit testing server.c will be much easier when server.c will not depend on the monitor code..

2 2

[PATCH] sysdb_get_user_attr: use fqn for subdomain users
by Pavel Březina 09 Oct '14

09 Oct '14

Without this patch IFP is unable to find subdomain users.

3 4

[PATCH] MAN: Build the sss_rpcidmapd man page conditionally
by Jakub Hrozek 09 Oct '14

09 Oct '14

Hi, Joschi Brauchle reported via e-mail that we errorneously built the rpcidmap man page even when SSSD was configured without the NFS plugin completely.

2 2

[PATCH] DP: Print a type as hexadecimal number in debug message.
by Lukas Slebodnik 09 Oct '14

09 Oct '14

ehlo, I was analysing log files for unrelated problem and I had to convert decimal number to hexidecimalt to compare type in debug message in function be_get_account_info. BE_REQ_* numbers are defined as hexadecimal in header file src/providers/data_provider.h Simple patch is attached. LS

3 2

[PATCH] Suppress warning maybe-uninitialized
by Lukas Slebodnik 07 Oct '14

07 Oct '14

ehlo, There is a warning with some version of gcc (4.9.1, 4.4.7). Warning is not visible with gcc 4.8 In my opinion, variable "a" should be initialized every time if "maps" is not NULL. src/providers/ldap/sdap.c: In function 'sdap_parse_entry': src/providers/ldap/sdap.c:481:56: warning: 'a' may be used uninitialized in this function [-Wmaybe-uninitialized] for (ai = a; ai < attrs_num; ai++) { ^ src/providers/ldap/sdap.c:307:9: note: 'a' was declared here int a, i, ret, ai; ^ CCLD libsss_ldap_common.la You can also see warnin in koji build on fedora 21 https://kojipkgs.fedoraproject.org//packages/sssd/1.12.1/2.fc21/data/logs/x… If you have better idea how to fix this warning I will be glad to review your patch. LS

3 7

[PATCH] CI: Allow running outside source root
by Nikolai Kondrashov 07 Oct '14

07 Oct '14

Hi everyone, The attached patch is based on Lukas's work on his "CI: Improve usability of ci script" patch: https://lists.fedorahosted.org/pipermail/sssd-devel/2014-September/021456.h… It simplifies some things, and does some of them in the way I proposed, but Lukas didn't agree to. I'm creating a separate thread, so we can perhaps take a fresh look at this issue, without the baggage of the old one, and maybe finally agree on a solution. Nick

1 0

[PATCH] BUILD: Use $(MKDIR_P) in Makefile.am
by Jakub Hrozek 06 Oct '14

06 Oct '14

Hi, While I was talking to the Fedora automake maintainer about an enhancement related to test environment, he suggested to make the change in the attached patch.

3 5

[PATCH] MONITOR: Make internal functions static
by Jakub Hrozek 06 Oct '14

06 Oct '14

Please see the attached trivial patch.

2 2

← Newer
1
2
3
4
5
6
7
8
9
10
Older →

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

sssd-devel October 2014