Bug 676689 - crash while adding a new user to be synced to windows
crash while adding a new user to be synced to windows
Product: 389
Classification: Community
Component: Sync Service (Show other bugs)
Unspecified Unspecified
high Severity high
: ---
: ---
Assigned To: Rich Megginson
Viktor Ashirov
Depends On:
Blocks: 639035 389_1.2.8 676733
  Show dependency treegraph
Reported: 2011-02-10 14:34 EST by Rich Megginson
Modified: 2015-12-07 11:41 EST (History)
1 user (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 676733 (view as bug list)
Last Closed: 2015-12-07 11:41:17 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
0001-Bug-676689-crash-while-adding-a-new-user-to-be-synce.patch (2.16 KB, patch)
2011-02-10 17:28 EST, Rich Megginson
nkinder: review+
Details | Diff

  None (edit)
Description Rich Megginson 2011-02-10 14:34:30 EST
Description of problem:

I set up windows sync (2008 R2).  Initial sync went ok.  I added an ntUser to the DS side - when it went to sync it crashed with an assertion abort in ldap_next_entry() in windows_search_entry_ext().  It seems that openldap ldap_next_entry(LDAP *, LDAPMessage *entry) does not like entry == NULL - it asserts and aborts, while mozldap ldap_next_entry will just return NULL in that case.
Comment 1 Rich Megginson 2011-02-10 17:28:04 EST
Created attachment 478135 [details]
Comment 2 Rich Megginson 2011-02-10 18:40:08 EST
To ssh://git.fedorahosted.org/git/389/ds.git
   96bb64a..8a15fd4  389-ds-base-1.2.8 -> 389-ds-base-1.2.8
commit 8a15fd460396e7d04a7907d4948493afba2b8083
Author: Rich Megginson <rmeggins@redhat.com>
Date:   Thu Feb 10 15:23:20 2011 -0700
    Reviewed by: nkinder (Thanks!)
    Branch: 389-ds-base-1.2.8
    Fix Description: The OpenLDAP ldap_next_entry() function will assert and
    abort if passed a NULL message.  Mozldap ldap_next_entry() will just return
    NULL.  Fix the server to not pass NULL to ldap_next_entry().
    Platforms tested: RHEL6 x86_64
    Flag Day: no
    Doc impact: no
    (cherry picked from commit 82b362176a8874ace0fd53e69ca6838d33a4881a)
commit 82b362176a8874ace0fd53e69ca6838d33a4881a
Author: Rich Megginson <rmeggins@redhat.com>
Date:   Thu Feb 10 15:23:20 2011 -0700
    Branch: master
Comment 3 Amita Sharma 2011-05-03 02:25:33 EDT
[root@rheltest etc]# ldapadd -x -D "cn=Directory Manager" -w abc -h localhost
-p 389 << EOF
> dn: cn=testuser,ou=users,ou=unix,dc=corp,dc=example,dc=com
> objectclass: person
> objectClass: inetorgperson
> objectclass: ntUser
> sn: User
> givenName: Test
> ntUserDomainId: testuser
> ntUserCreateNewAccount: TRUE
adding new entry "cn=testuser,ou=users,ou=unix,dc=corp,dc=example,dc=com"

- Entry replicated to AD and no crash found.
hence verified.

Note You need to log in before you can comment on or make changes to this bug.