Bug 305121 - Server hangs when adding a group with two password entries
Server hangs when adding a group with two password entries
Product: 389
Classification: Community
Component: Directory Server (Show other bugs)
All Linux
high Severity high
: ---
: ---
Assigned To: Rich Megginson
Viktor Ashirov
Depends On:
Blocks: 240316 FDS1.1.0
  Show dependency treegraph
Reported: 2007-09-25 10:23 EDT by Miloslav Trmač
Modified: 2015-12-07 11:35 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2015-12-07 11:35:02 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
diffs (1.30 KB, patch)
2007-10-08 23:40 EDT, Rich Megginson
no flags Details | Diff

  None (edit)
Description Miloslav Trmač 2007-09-25 10:23:08 EDT
Version-Release number of selected component (if applicable):

Steps to Reproduce:
1. Set up a DS instance using /usr/sbin/setup-ds.pl
2. ldapadd something like the following:
dn: cn=my_group,ou=Groups,dc=ROOT
objectClass: posixGroup
cn: my_group
gidNumber: 424242
userPassword: unknown1
userPassword: unknown2
Actual results:
ldapadd never receives a reply from the server, the server becomes unresponsive
and must be killed with SIGKILL.

Expected results:
Creation of the specified group, or perhaps an error message.
Comment 1 Rich Megginson 2007-10-08 23:40:55 EDT
Created attachment 220601 [details]
Comment 2 Rich Megginson 2007-10-09 13:52:48 EDT
Checking in pw.c;
/cvs/dirsec/ldapserver/ldap/servers/slapd/pw.c,v  <--  pw.c
new revision: 1.16; previous revision: 1.15

Reviewed by: nhosoi,nkinder (Thanks!)
Files: see diff
Branch: HEAD
Fix Description: The pw_encodevals() was not encoding each value, only the first
one, then setting each new value to the same encoded value.  The solution is to
move char *enc into the loop so that it is allocated anew each time.
Platforms tested: RHEL5 x86_64
Flag Day: no
Doc impact: no

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