The error result text message should be obtained just prior to sending result

[389-ds-bot]

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/47817

• Created at 2014-06-18 03:36:21 by mreynolds (@mreynolds389)
• Closed as Fixed
• Assigned to mreynolds (@mreynolds389)
• Associated bugzillas
  • https://bugzilla.redhat.com/show_bug.cgi?id=1118082

---

Its possible that backend plugins can set the error text, which frees the existing result text(if any). The problem is that in the backend we grab a pointer to the result text before we call the backend postop plugins. If a plugin sets the result text we will get invalid read errors reported by valgrind.

[389-ds-bot]

Comment from mreynolds (@mreynolds389) at 2014-06-18 23:15:49

attachment
0001-Ticket-47817-The-error-result-text-message-should-be.patch

[389-ds-bot]

Comment from mreynolds (@mreynolds389) at 2014-06-19 01:45:02

git merge ticket47817
Updating f2eb45b..68e7668
Fast-forward
ldap/servers/slapd/back-ldbm/ldbm_add.c | 6 ++++--
ldap/servers/slapd/back-ldbm/ldbm_delete.c | 3 ++-
ldap/servers/slapd/back-ldbm/ldbm_modify.c | 1 +
ldap/servers/slapd/back-ldbm/ldbm_modrdn.c | 1 +
4 files changed, 8 insertions(+), 3 deletions(-)

git push origin master
f2eb45b..68e7668 master -> master

commit 68e7668
Author: Mark Reynolds mreynolds389@redhat.com
Date: Tue Jun 17 17:58:54 2014 -0400

887a72e..dd51540 389-ds-base-1.3.2 -> 389-ds-base-1.3.2
commit dd51540

98b0853..e6762cb 389-ds-base-1.3.1 -> 389-ds-base-1.3.1
commit e6762cb8cd43d8753f64f00cb3da36dc817d3c5d

d8f6af4..6f6142e 389-ds-base-1.2.11 -> 389-ds-base-1.2.11
commit 6f6142e

[389-ds-bot]

Comment from nhosoi (@nhosoi) at 2014-07-10 06:34:46

Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1118082

[389-ds-bot]

Comment from mreynolds (@mreynolds389) at 2017-02-11 23:11:28

Metadata Update from @mreynolds389:

• Issue assigned to mreynolds389
• Issue set to the milestone: 1.2.11.30