Skip to content

Issue 6417 - (3rd) If an entry RDN is identical to the suffix, then E…#6480

Merged
tbordaz merged 1 commit into
389ds:mainfrom
tbordaz:Issue_6417_3rd
Jan 13, 2025
Merged

Issue 6417 - (3rd) If an entry RDN is identical to the suffix, then E…#6480
tbordaz merged 1 commit into
389ds:mainfrom
tbordaz:Issue_6417_3rd

Conversation

@tbordaz
Copy link
Copy Markdown
Contributor

@tbordaz tbordaz commented Jan 8, 2025

…ntryrdn gets broken during a reindex

Bug description:
The previous fix had a flaw.
In case entryrdn_lookup_dn is called with an undefined suffix
the lookup of the suffix trigger a crash.
For example it can occur during internal search of an
unexisting map (view plugin).
The issue exists in all releases but is hidden since 2.3.

Fix description:
testing the suffix is defined

fixes: #6417

Reviewed by:

@tbordaz tbordaz requested a review from progier389 January 8, 2025 11:15
Copy link
Copy Markdown
Contributor

@progier389 progier389 left a comment

Choose a reason for hiding this comment

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

LGTM

…ntryrdn gets broken during a reindex

Bug description:
	The previous fix had a flaw.
	In case entryrdn_lookup_dn is called with an undefined suffix
	the lookup of the suffix trigger a crash.
	For example it can occur during internal search of an
	unexisting map (view plugin).
	The issue exists in all releases but is hidden since 2.3.

Fix description:
	testing the suffix is defined

fixes: 389ds#6417

Reviewed by: Pierre Rogier (THnaks !)
@tbordaz tbordaz merged commit ea7de48 into 389ds:main Jan 13, 2025
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
	The previous fix had a flaw.
	In case entryrdn_lookup_dn is called with an undefined suffix
	the lookup of the suffix trigger a crash.
	For example it can occur during internal search of an
	unexisting map (view plugin).
	The issue exists in all releases but is hidden since 2.3.

Fix description:
	testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
	The previous fix had a flaw.
	In case entryrdn_lookup_dn is called with an undefined suffix
	the lookup of the suffix trigger a crash.
	For example it can occur during internal search of an
	unexisting map (view plugin).
	The issue exists in all releases but is hidden since 2.3.

Fix description:
	testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
	The previous fix had a flaw.
	In case entryrdn_lookup_dn is called with an undefined suffix
	the lookup of the suffix trigger a crash.
	For example it can occur during internal search of an
	unexisting map (view plugin).
	The issue exists in all releases but is hidden since 2.3.

Fix description:
	testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
tbordaz added a commit that referenced this pull request Jan 13, 2025
…ntryrdn gets broken during a reindex (#6480)

Bug description:
        The previous fix had a flaw.
        In case entryrdn_lookup_dn is called with an undefined suffix
        the lookup of the suffix trigger a crash.
        For example it can occur during internal search of an
        unexisting map (view plugin).
        The issue exists in all releases but is hidden since 2.3.

Fix description:
        testing the suffix is defined

fixes: #6417

Reviewed by: Pierre Rogier (THnaks !)
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#6480)"

This reverts commit 5d31b9b.
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