diff options
| author | Rich Megginson <rmeggins@redhat.com> | 2009-02-13 19:41:26 +0000 |
|---|---|---|
| committer | Rich Megginson <rmeggins@redhat.com> | 2009-02-13 19:41:26 +0000 |
| commit | 1551ce2a6f91b7811e49fed8497b399263a5310f (patch) | |
| tree | 985c0ea64f15749f6d24cdf11325c46e9cb3b692 | |
| parent | ce5ecdab7c09c4487f7857a916568b47981d3272 (diff) | |
| download | ds-1551ce2a6f91b7811e49fed8497b399263a5310f.tar.gz ds-1551ce2a6f91b7811e49fed8497b399263a5310f.tar.xz ds-1551ce2a6f91b7811e49fed8497b399263a5310f.zip | |
Resolves: bug 431607
Description: referential integrity plugin incorrectly treats "rename" operations
Fix Description: join RDNs with "," instead of ", " to make the new DN - thanks to Andrey Ivanov (andrey.ivanov@polytechnique.fr)
| -rw-r--r-- | ldap/servers/plugins/referint/referint.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/ldap/servers/plugins/referint/referint.c b/ldap/servers/plugins/referint/referint.c index eee69957..25df199f 100644 --- a/ldap/servers/plugins/referint/referint.c +++ b/ldap/servers/plugins/referint/referint.c @@ -401,8 +401,8 @@ int update_integrity(char **argv, char *origDN, char *newrDN, int logChanges){ dnsize = 0; for(x=1; dnParts[x] != NULL; x++) { - /* +2 for space and comma adding later */ - dnsize += strlen(dnParts[x]) + 2; + /* +1 for comma adding later */ + dnsize += strlen(dnParts[x]) + 1; } /* add the newrDN length */ dnsize += strlen(newrDN) + 1; @@ -411,7 +411,7 @@ int update_integrity(char **argv, char *origDN, char *newrDN, int logChanges){ strcat(newDN, newrDN); for(x=1; dnParts[x] != NULL; x++) { - strcat(newDN, ", "); + strcat(newDN, ","); strcat(newDN, dnParts[x]); } |