ldap/servers/plugins/dna/dna.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commits: commit d8c6e0c5d06f38b8f75f72d7cb44a3a8753b6301 Author: Nathan Kinder nkinder@redhat.com Date: Tue Oct 1 08:54:32 2013 -0700
Ticket 47539 - Disabling DNA plug-in throws error 53
If the DNA plug-in is enabled and any modify is attempted against the main DNA plug-in config entry (like disabling it), the operation will be rejected with error 53.
The plug-in config validation function attempts to skip validation of the main DNA plug-in config entry, but it returns DNA_FAILURE, which causes the operation to be rejected. The validation function needs to return DNA_SUCCESS to allow changes to be made to the main config entry.
diff --git a/ldap/servers/plugins/dna/dna.c b/ldap/servers/plugins/dna/dna.c index 90a66c6..9794a4a 100644 --- a/ldap/servers/plugins/dna/dna.c +++ b/ldap/servers/plugins/dna/dna.c @@ -758,7 +758,7 @@ dna_parse_config_entry(Slapi_PBlock *pb, Slapi_Entry * e, int apply) /* If this is the main DNA plug-in * config entry, just bail. */ if (strcasecmp(getPluginDN(), slapi_entry_get_ndn(e)) == 0) { - ret = DNA_FAILURE; + ret = DNA_SUCCESS; goto bail; }
389-commits@lists.fedoraproject.org