On Tue, 2012-06-19 at 17:15 +0200, Jan Zelený wrote: > This patch fixes an issue which resulted in a need to initialize > responder with data from local domain, otherwise it would not correctly > detect requests for subdomains. Similar situation can occur if new > subdomain is added at runtime. > > The solution is to ask for a list of subdomains in case there is a > candidate domain identified in the process of matching re_expressions > with given name. >
Nack You're copy-pasting the same bit of code around sss_dp_get_domains_send() in quite a few places. Could you just wrap these all in a common function and avoid all the reuse, please? Also, I don't understand the point of the pam_forwarder_cb() changes. Were those supposed to be a different patch?
signature.asc
Description: This is a digitally signed message part
_______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel