Attachment 299482 Details for Bug 439097 – CVS Diffs

[patch] CVS Diffs

diffs.txt (text/plain), 1.91 KB, created by Nathan Kinder on 2008-03-28 15:31:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nathan Kinder

Created: 2008-03-28 15:31:27 UTC

Size: 1.91 KB

patch

obsolete

>Index: memberof.c
>===================================================================
>RCS file: /cvs/dirsec/ldapserver/ldap/servers/plugins/memberof/memberof.c,v
>retrieving revision 1.2
>diff -u -5 -t -r1.2 memberof.c
>--- memberof.c	26 Mar 2008 22:25:49 -0000	1.2
>+++ memberof.c	28 Mar 2008 15:18:19 -0000
>@@ -652,12 +652,23 @@
>                                                 break;
>                                         }
>                                 
>                                 case LDAP_MOD_DELETE:
>                                         {
>-                                                /* remove group DN from targets */
>-                                                memberof_del_smod_list(pb, dn, smod);
>+                                                /* If there are no values in the smod, we should
>+                                                 * just do a replace instead.  The  user is just
>+                                                 * trying to delete all members from this this
>+                                                 * entry, which the replace code deals with. */
>+                                                if (slapi_mod_get_num_values(smod) == 0)
>+                                                {
>+                                                        memberof_replace_list(pb, dn);
>+                                                }
>+                                                else
>+                                                {
>+                                                        /* remove group DN from target values in smod*/
>+                                                        memberof_del_smod_list(pb, dn, smod);
>+                                                }
>                                                 break;
>                                         }
> 
>                                 case LDAP_MOD_REPLACE:
>                                         {

Actions: View | Diff

Attachments on bug 439097: 299482