From 1eeb4bc0ffb7b470c255ca8888a752122d551d1d Mon Sep 17 00:00:00 2001 From: Michael Adam Date: Fri, 11 Apr 2014 21:03:06 +0200 Subject: [PATCH] idmap_rid: remove a legacy comment from sid_to_id With the introduction of the ID_TYPE_BOTH mapping to idmap_rid, it is not a deficiency but a virtue of the rid backend that it does not care about the existence or type of the sid to be mapped. Signed-off-by: Michael Adam Reviewed-by: Stefan Metzmacher --- source3/winbindd/idmap_rid.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/source3/winbindd/idmap_rid.c b/source3/winbindd/idmap_rid.c index 0856c802375..099f68c5ee3 100644 --- a/source3/winbindd/idmap_rid.c +++ b/source3/winbindd/idmap_rid.c @@ -87,10 +87,6 @@ static NTSTATUS idmap_rid_id_to_sid(struct idmap_domain *dom, struct id_map *map sid_compose(map->sid, &domain->sid, map->xid.id - dom->low_id + ctx->base_rid); - /* We **really** should have some way of validating - the SID exists and is the correct type here. But - that is a deficiency in the idmap_rid design. */ - map->status = ID_MAPPED; map->xid.type = ID_TYPE_BOTH; -- 2.34.1