If you run ds_newinst.pl with a bare minimum .inf file, you will see the following aci created in cn=config: aci: (targetattr="*")(version 3.0; acl "Configuration Administrator"; allow (a ll) userdn="ldap:///uid=,ou=Administrators, ou=TopologyManagement, o=Netscap eRoot";) We should not create this aci, as we do not have a configuration DS if we are creating a minimal fedora-ds-base.
Created attachment 148687 [details] diffs
Looks good.
Created attachment 148697 [details] cvs commit log Reviewed by: nhosoi (Thanks!) Files: see diff Branch: HEAD Fix Description: Unknown to me until just now, PL_strdup(NULL) will return "" - the empty string. The code in config_suitespot() expects that empty or unused fields are NULL. The solution is to create a create_instance_strdup() wrapper around PL_strdup() and use that in cases where the argument may be NULL. I checked create_instance.c. Every other place where PL_strdup is used, the argument is checked for NULL first. So these are the only places affected. Instance creation works fine after this change and does not create the offending aci. Platforms tested: RHEL4 Flag Day: no Doc impact: no
ds_newinst.pl has been discard in the new 8.0 release. However, related code still used. Bug verification test has been done. Please refer to bug 229824 for details.