URL: https://github.com/SSSD/sssd/pull/202
Title: #202: T3315 infopipe group users master
mzidek-rh commented:
"""
Actually, I looking at it again I do not think these patches solve the issue.
The reproducer is not correct. After you update all the members in the group
with dbus call in your reproducer, the user is added to the cache (as a normal
user, not as a ghost user)... and this is what worked even before your
paches... so it does not seem to be fixed when only ghost users are stored (and
that is what the bug was about).
Also there is a warning:
/home/user/gitrepo/sssd/src/responder/ifp/ifp_groups.c: In function
‘resolv_ghosts_step’:
/home/user/gitrepo/sssd/src/responder/ifp/ifp_groups.c:600:37: warning: passing
argument 3 of ‘sss_parse_internal_fqname’ from incompatible pointer type
[-Wincompatible-pointer-types]
&ghost_name, NULL);
^
In file included from
/home/user/gitrepo/sssd/src/responder/ifp/ifp_groups.c:24:0:
/home/user/gitrepo/sssd/src/util/util.h:284:9: note: expected ‘char **’ but
argument is of type ‘const char **’
errno_t sss_parse_internal_fqname(TALLOC_CTX *mem_ctx,
^~~~~~~~~~~~~~~~~~~~~~~~~
"""
See the full comment at
https://github.com/SSSD/sssd/pull/202#issuecomment-290713169
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
