----- Original Message ----- > 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?
Well I suppose I can do that but the gain would not be that significant since all I do is basically error checking which would still need to be there. How about some kind of macro with one argument: callback name. Would that be acceptable? > > Also, I don't understand the point of the pam_forwarder_cb() changes. > Were those supposed to be a different patch? No, that is actually necessary because of the re_expression matching that is done in pam_parse_in_data* Thanks for the review Jan _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel