Bug 185477 - ldif2db allows entries without a parent to be imported
Summary: ldif2db allows entries without a parent to be imported
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Red Hat Directory Server
Classification: Red Hat
Component: Command Line Utilities
Version: 7.1
Hardware: All
OS: Linux
medium
medium
Target Milestone: DS8.0
: ---
Assignee: Nathan Kinder
QA Contact: Orla Hegarty
URL:
Whiteboard:
Depends On:
Blocks: 159328 182367
TreeView+ depends on / blocked
 
Reported: 2006-03-15 00:21 UTC by Orla Hegarty
Modified: 2008-08-11 23:45 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-05-26 20:11:50 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
CVS Diffs (847 bytes, patch)
2006-03-15 00:25 UTC, Nathan Kinder 		no flags Details | Diff
cvs commit idl_new.c (HEAD and Directory71RtmBranch) (1.35 KB, text/plain)
2006-03-15 00:51 UTC, Noriko Hosoi 		no flags Details
cvs diff ldif2ldbm.c and the commit message (1.70 KB, text/plain)
2006-03-16 02:59 UTC, Noriko Hosoi 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2006:0270 0 normal SHIPPED_LIVE Moderate: Red Hat Directory Server 7.1 security update 2006-05-17 04:00:00 UTC

Comment 1 Nathan Kinder 2006-03-15 00:25:56 UTC 
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.
Comment 2 Noriko Hosoi 2006-03-15 00:51:40 UTC 
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.
Comment 3 Orla Hegarty 2006-03-15 01:19:46 UTC 
*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.
Comment 4 Noriko Hosoi 2006-03-16 02:59:18 UTC 
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!!!)
Comment 5 Orla Hegarty 2006-03-16 21:49:57 UTC 
New testcases added to import test plan to verify this bug. Verified fixed
against all supported platforms for DS 7.1 SP 2
Comment 7 Orla Hegarty 2006-05-26 17:49:07 UTC 
Somehow the errata system did not automatically close these bugs even though DS
SP 2 is shipped and available live on RHN
Comment 8 Orla Hegarty 2006-05-26 17:52:27 UTC 
trying to manually close
Comment 9 Orla Hegarty 2006-05-26 20:11:50 UTC 
trying again
Comment 10 Chandrasekar Kannan 2007-07-25 19:19:30 UTC 
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.
Comment 11 Chandrasekar Kannan 2008-08-11 23:45:08 UTC 
Bug already CLOSED. setting screened+ flag
Note You need to log in before you can comment on or make changes to this bug.