diff options
| author | Sumit Bose <sbose@redhat.com> | 2015-02-24 18:32:43 +0100 |
|---|---|---|
| committer | Tomas Babej <tbabej@redhat.com> | 2015-02-24 22:47:36 +0100 |
| commit | 1a37822c3a80ffc05e28f5b9f1bacd248f5f2375 (patch) | |
| tree | 288ac39699811059ff53ee31e86a55a9578ccfe8 /daemons | |
| parent | 93f3bb3ddd5b93a5eb731239d2c03c0a350367ca (diff) | |
| download | freeipa-1a37822c3a80ffc05e28f5b9f1bacd248f5f2375.tar.gz freeipa-1a37822c3a80ffc05e28f5b9f1bacd248f5f2375.tar.xz freeipa-1a37822c3a80ffc05e28f5b9f1bacd248f5f2375.zip | |
ipa-range-check: do not treat missing objects as error
Currently the range check plugin will return a 'Range Check error'
message if a ldapmodify operation tries to change a non-existing object.
Since the range check plugin does not need to care about non-existing
objects we can just return 0 indicating that the range check plugin has
done its work.
Resolves https://fedorahosted.org/freeipa/ticket/4924
Reviewed-By: Tomas Babej <tbabej@redhat.com>
Diffstat (limited to 'daemons')
| -rw-r--r-- | daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c b/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c index 1fd543c18..5b53a2fe5 100644 --- a/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c +++ b/daemons/ipa-slapi-plugins/ipa-range-check/ipa_range_check.c @@ -545,8 +545,9 @@ static int ipa_range_check_pre_op(Slapi_PBlock *pb, int modtype) ret = slapi_search_internal_get_entry(dn, NULL, &entry, ctx->plugin_id); if (ret != 0 || entry == NULL) { - LOG_FATAL("Missing entry to modify.\n"); - ret = LDAP_NO_SUCH_OBJECT; + LOG("Missing entry to modify.\n"); + /* No range object, nothing to do. */ + ret = 0; goto done; } free_entry = true; |