This might not be used a lot and might not survive for long, but at least it

should work as expected :-)

Fix wb_delgrpmember.

Volker

diff --git a/source/nsswitch/wbinfo.c b/source/nsswitch/wbinfo.c
index 4f6cc4225a7825232af8c61f9ed77556c8eb733b..772332ee59272fe981fb1675c4592765a0ad0cec 100644 (file)
--- a/source/nsswitch/wbinfo.c
+++ b/source/nsswitch/wbinfo.c
@@ -1179,7 +1179,7 @@ int main(int argc, char **argv)
                        break;
                case 'O':
                        if ( !wbinfo_remove_user_from_group(string_arg) ) {
-                               d_printf("Could not remove user kfrom group\n");
+                               d_printf("Could not remove user from group\n");
                                goto done;
                        }
                        break;

diff --git a/source/nsswitch/winbindd_acct.c b/source/nsswitch/winbindd_acct.c
index 2f94a2efd0bed24c1e037038357383b2a88bf449..be3f1405e433ca0ae08aaa628799023250fc96cf 100644 (file)
--- a/source/nsswitch/winbindd_acct.c
+++ b/source/nsswitch/winbindd_acct.c
@@ -371,7 +371,7 @@ static WINBINDD_GR* string2group( char *string )
                if ( num_gr_members ) {
                        fstring buffer;
                        
-                       gr_members = (char**)smb_xmalloc(sizeof(char*)*num_gr_members+1);
+                       gr_members = (char**)smb_xmalloc(sizeof(char*)*(num_gr_members+1));
                        
                        i = 0;
                        while ( next_token(&str, buffer, ",", sizeof(buffer)) && i<num_gr_members ) {
@@ -815,9 +815,11 @@ static BOOL wb_delgrpmember( WINBINDD_GR *grp, const char *user )
        if ( !grp || !user )
                return False;
        
-       for ( i=0; i<grp->num_gr_mem && !found; i++ ) {
-               if ( StrCaseCmp( grp->gr_mem[i], user ) == 0 ) 
+       for ( i=0; i<grp->num_gr_mem; i++ ) {
+               if ( StrCaseCmp( grp->gr_mem[i], user ) == 0 ) {
                        found = True;
+                       break;
+               }
        }
        
        if ( !found ) 
@@ -826,8 +828,10 @@ static BOOL wb_delgrpmember( WINBINDD_GR *grp, const char *user )
        /* still some remaining members */
 
        if ( grp->num_gr_mem > 1 ) {
-               memmove( grp->gr_mem[i], grp->gr_mem[i+1], sizeof(char*)*(grp->num_gr_mem-(i+1)) );
+               SAFE_FREE(grp->gr_mem[i]);
                grp->num_gr_mem--;
+               grp->gr_mem[i] = grp->gr_mem[grp->num_gr_mem];
+               grp->gr_mem[grp->num_gr_mem] = NULL;
        }
        else {  /* last one */
                free_winbindd_gr( grp );
@@ -1237,7 +1241,7 @@ enum winbindd_result winbindd_remove_user_from_group(struct winbindd_cli_state *
        group = state->request.data.acct_mgt.groupname;
        user = state->request.data.acct_mgt.username;
        
-       DEBUG(3, ("[%5lu]:  remove_user_to_group: delete %s from %s\n", (unsigned long)state->pid, 
+       DEBUG(3, ("[%5lu]:  remove_user_from_group: delete %s from %s\n", (unsigned long)state->pid, 
                user, group));
        
        /* don't worry about checking the username since we're removing it anyways */
@@ -1245,7 +1249,7 @@ enum winbindd_result winbindd_remove_user_from_group(struct winbindd_cli_state *
        /* make sure it is a valid group */
        
        if ( !(grp = wb_getgrnam( group )) ) {
-               DEBUG(4,("winbindd_remove_user_to_group: Cannot remove a user to a non-extistent group\n"));
+               DEBUG(4,("winbindd_remove_user_from_group: Cannot remove a user from a non-extistent group\n"));
                return WINBINDD_ERROR;  
        }