<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
5th patch)
<blockquote cite="mid:20150603153650.GS2805@hendrix" type="cite">
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">+static void sdap_inherit_user_options(struct sss_domain_info *parent_dom,
+ struct sdap_attr_map
*parent_user_map,
+ struct sdap_attr_map
*child_user_map)
+{
+ int inherit_options[] = {
+ SDAP_AT_USER_PRINC,
+ SDAP_OPTS_USER /* sentinel */
+ };
+ int i;
+
+ for (i = 0; inherit_options[i] != SDAP_OPTS_USER; i++) {
+ sdap_copy_map_entry(parent_user_map, child_user_map, i);
</pre>
</blockquote>
<pre wrap="">s/i/inherit_options[i] ?
</pre>
</blockquote>
<pre wrap="">
No, the parameter is an integer.
</pre>
</blockquote>
Yes, it's integer - I'd expect that you want to call:<br>
<br>
<pre wrap="">sdap_copy_map_entry(parent_user_map, child_user_map, SDAP_AT_USER_PRINC) rather then sdap_copy_map_entry(parent_user_map, child_user_map, 0)
I don't see what's the point of values of inherit_options at all if the code would be right...I'll have another look tomorrow with fresh eyes.
Jakub it must be a typo - I can't resolve user from subdomain with patches applied, but if I do the proposed change resolving of subdomain users starts to work.
</pre>
<br>
<blockquote cite="mid:20150603153650.GS2805@hendrix" type="cite">
<pre wrap="">
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
sssd-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
</pre>
</blockquote>
<br>
</body>
</html>