Skip to content

Issue 6417 - (2nd) If an entry RDN is identical to the suffix, then E…#6460

Merged
tbordaz merged 1 commit into
389ds:mainfrom
tbordaz:Issue_6417_2nd
Jan 6, 2025
Merged

Issue 6417 - (2nd) If an entry RDN is identical to the suffix, then E…#6460
tbordaz merged 1 commit into
389ds:mainfrom
tbordaz:Issue_6417_2nd

Conversation

@tbordaz

@tbordaz tbordaz commented Dec 20, 2024

Copy link
Copy Markdown
Contributor

…ntryrdn gets broken during a reindex

Bug description:
The primary fix has a flaw as it assumes that the
suffix ID is '1'.
If the RUV entry is the first entry of the database
the server loops indefinitely

Fix description:
Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (reviewed the first fix)

Comment thread ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c Outdated
Comment thread ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c
…ntryrdn gets broken during a reindex

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: 389ds#6417

Reviewed by: Pierre Rogier (also reviewed the first fix)

@progier389 progier389 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tbordaz tbordaz merged commit a79887e into 389ds:main Jan 6, 2025
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 6, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
tbordaz added a commit that referenced this pull request Jan 7, 2025
…ntryrdn gets broken during a reindex (#6460)

Bug description:
	The primary fix has a flaw as it assumes that the
	suffix ID is '1'.
	If the RUV entry is the first entry of the database
	the server loops indefinitely

Fix description:
	Read the suffix ID from the entryrdn index

fixes: #6417

Reviewed by: Pierre Rogier (also reviewed the first fix)
Firstyear added a commit to Firstyear/389-ds-base that referenced this pull request Sep 25, 2025
…, then Entryrdn gets broken during a reindex (389ds#6460)"

This reverts commit 0fa34c1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

If an entry RDN is identical to the suffix, then Entryrdn gets broken during a reindex

2 participants