r18775: Performing an ldb op of 'do nothing' is pointless, and breaks against

OpenLDAP.

Andrew Bartlett
(This used to be commit 9ce88a8917d383104c47f794a8c554c43d13e383)

diff --git a/source4/rpc_server/samr/dcesrv_samr.c b/source4/rpc_server/samr/dcesrv_samr.c
index 971541bf9d47dfa522dc18d1ac6fe80b469cf5c9..4a73938a8e20328e52520e5225d0cbd4279827c2 100644 (file)
--- a/source4/rpc_server/samr/dcesrv_samr.c
+++ b/source4/rpc_server/samr/dcesrv_samr.c
@@ -2038,7 +2038,7 @@ static NTSTATUS samr_SetGroupInfo(struct dcesrv_call_state *dce_call, TALLOC_CTX
                break;
        case GROUPINFOATTRIBUTES:
                /* This does not do anything obviously visible in W2k3 LDAP */
-               break;
+               return NT_STATUS_OK;
        default:
                return NT_STATUS_INVALID_INFO_CLASS;
        }