On Fri, Jan 24, 2014 at 01:20:12PM +0100, Lukas Slebodnik wrote:
>Looks like you forgot to include some changes?
>
>/home/remote/jhrozek/devel/sssd/src/providers/ipa/ipa_subdomains_id.c:393:15: error:
implicit declaration of function 'expand_homedir_template' is invalid in C99
[-Werror,-Wimplicit-function-declaration]
> homedir = expand_homedir_template(tmp_ctx, dom->subdomain_homedir, name,
> ^
>/home/remote/jhrozek/devel/sssd/src/providers/ipa/ipa_subdomains_id.c:393:13:
warning: incompatible integer to pointer conversion assigning to 'char *' from
'int' [-Wint-conversion]
> homedir = expand_homedir_template(tmp_ctx, dom->subdomain_homedir, name,
>
>/home/remote/jhrozek/devel/sssd/src/providers/ipa/ipa_subdomains_id.c:414:31:
warning: passing 'const struct sss_domain_info *' to parameter of type 'struct
sss_domain_info *' discards qualifiers
> [-Wincompatible-pointer-types-discards-qualifiers]
> ret = sysdb_set_user_attr(dom, fqname, attrs, SYSDB_MOD_REP);
> ^~~
>/home/remote/jhrozek/devel/sssd/src/db/sysdb.h:558:49: note: passing argument to
parameter 'domain' here
>int sysdb_set_user_attr(struct sss_domain_info *domain,
> ^
>/home/remote/jhrozek/devel/sssd/src/providers/ipa/ipa_subdomains_id.c:373:9: warning:
variable 'attrs' is uninitialized when used here [-Wuninitialized]
> if (attrs == NULL) {
> ^~~~~
>/home/remote/jhrozek/devel/sssd/src/providers/ipa/ipa_subdomains_id.c:363:30: note:
initialize the variable 'attrs' to silence this warning
> struct sysdb_attrs *attrs;
> ^
> = NULL
>
>
It is well know problem with new gcc
https://bugzilla.redhat.com/show_bug.cgi?id=986923.
LS
Yeah, this bug pretty much made me switch to clang on my devel machine..
(I still use gcc on VMs, though, so most patches get compiled with both)