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