Here is what I intend to implement
http://port389.org/wiki/Configuring_scope_for_referential_integrity_plugin

attachment
0001-Ticket-47621-make-referential-integrity-configuratio.patch

{{{
plugin_attr_values = slapi_entry_attr_get_charray(plugin_entry, "nsslapd-pluginEntryScope");
...
plugin_EntryScope[j++] = slapi_sdn_new_dn_passin(plugin_attr_values[i]);
}}}
still need to free plugin_attr_values but not the individual plugin_attr_values[i]. same with
{{{
plugin_attr_value = slapi_entry_attr_get_charptr(plugin_entry, "nsslapd-pluginExcludeEntryScope");
}}}
why byref here? passin is used above
{{{
plugin_ExcludeEntryScope = slapi_sdn_new_dn_byref(plugin_attr_value);
}}}

Replying to [comment:3 rmeggins]:

{{{
plugin_attr_values = slapi_entry_attr_get_charray(plugin_entry, "nsslapd-pluginEntryScope");
...
plugin_EntryScope[j++] = slapi_sdn_new_dn_passin(plugin_attr_values[i]);
}}}
still need to free plugin_attr_values but not the individual plugin_attr_values[i].

just missed it

same with

{{{
plugin_attr_value = slapi_entry_attr_get_charptr(plugin_entry, "nsslapd-pluginExcludeEntryScope");
}}}
it is either freed (if not valid dn) or passed in to plugin_ExcludeEntryScope
why byref here? passin is used above
{{{
plugin_ExcludeEntryScope = slapi_sdn_new_dn_byref(plugin_attr_value);
}}}

should be passin

attachment
0001-Ticket-47621-v2-make-referential-integrity-configura.patch

new patch attached, should address teh issues raised in review

$ git merge ticket47621
Updating 1119083..16dc94f
Fast-forward
ldap/servers/plugins/referint/referint.c | 86 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------
1 file changed, 71 insertions(+), 15 deletions(-)
$ git push origin master
Counting objects: 13, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 1.58 KiB, done.
Total 7 (delta 4), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
1119083..16dc94f master -> master

git patch file (389-ds-base-1.3.2) -- fixed the conflicts; passed the tests 100%
0001-Ticket-47621-v2-make-referential-integrity-configura.2.patch

Pushed to 389-ds-base-1.3.2:
19660fa..db3de3f 389-ds-base-1.3.2 -> 389-ds-base-1.3.2
commit db3de3f

Closing the ticket on behalf of Ludwig.

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1044173

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/958

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.