Bug 185477
| Summary: | ldif2db allows entries without a parent to be imported | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Red Hat Directory Server | Reporter: | Orla Hegarty <ohegarty> | ||||||||
| Component: | Command Line Utilities | Assignee: | Nathan Kinder <nkinder> | ||||||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | Orla Hegarty <ohegarty> | ||||||||
| Severity: | medium | Docs Contact: | |||||||||
| Priority: | medium | ||||||||||
| Version: | 7.1 | CC: | nhosoi, rmeggins, tngan | ||||||||
| Target Milestone: | DS8.0 | ||||||||||
| Target Release: | --- | ||||||||||
| Hardware: | All | ||||||||||
| OS: | Linux | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | Environment: | ||||||||||
| Last Closed: | 2006-05-26 20:11:50 UTC | Type: | --- | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | --- | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Bug Depends On: | |||||||||||
| Bug Blocks: | 159328, 182367 | ||||||||||
| Attachments: |
|
||||||||||
Created attachment 126136 [details]
cvs commit idl_new.c (HEAD and Directory71RtmBranch)
I've checked in the fix (idl_new.c) into CVS HEAD as well as
Directory71RtmBranch on behalf of Nathan.
Reviewed by Rich and Noriko.
*DOCS* In previous 7.X versions of Red Hat Directory Server, importing a malformed parent entry and some child entries resulted in the child entries being successfully added without a parent. This has been fixed. Created attachment 126192 [details]
cvs diff ldif2ldbm.c and the commit message
File:
ldif2ldbm.c
Change:
if returned idl is NULL, changed add_op_attrs to always set
IMPORT_ADD_OP_ATTRS_NO_PARENT in non-error case (err == 0 or err ==
DB_NOTFOUND).
Reviewed by Rich (Thank you!!!)
New testcases added to import test plan to verify this bug. Verified fixed against all supported platforms for DS 7.1 SP 2 Somehow the errata system did not automatically close these bugs even though DS SP 2 is shipped and available live on RHN trying to manually close trying again DS7.2 is not a valid milestone anymore. Anything thats set to DS7.2 should be set to DS8.0. Will make further changes per bug council on 07/24/2007, after this. Bug already CLOSED. setting screened+ flag |
Created attachment 126135 [details] CVS Diffs The problem is that the new IDL code is not exposing the DB_NOTFOUND error when the index key is not found. Import checks if the result was DB_NOTFOUND when checking for the parent entry's entrydn index. This works fine with the old IDL code.