It's actually OK to set this to BOTH. Signed-off-by: Jeff Layton <jlayton@xxxxxxxxx> --- cifsidmap.h | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/cifsidmap.h b/cifsidmap.h index e539719..656df86 100644 --- a/cifsidmap.h +++ b/cifsidmap.h @@ -45,10 +45,7 @@ struct cifs_sid { #define CIFS_UXID_TYPE_GID (2) /* mapping is a GID */ #define CIFS_UXID_TYPE_BOTH (3) /* usable as UID or GID */ -/* - * This struct represents both a uid or gid and its type. The type should - * never be set to CIFSIDMAP_BOTH. - */ +/* This struct represents a uid or gid and its type */ struct cifs_uxid { union { uid_t uid; -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-cifs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html