Commit 76ed23a5 authored by Eric W. Biederman's avatar Eric W. Biederman
Browse files

9p: Modify struct v9fs_session_info to use a kuids and kgids 



Change struct v9fs_session_info and the code that popluates it to use
kuids and kgids.  When parsing the 9p mount options convert the
dfltuid, dflutgid, and the session uid from the current user namespace
into kuids and kgids.  Modify V9FS_DEFUID and V9FS_DEFGUID to be kuid
and kgid values.

Cc: Eric Van Hensbergen <ericvh@gmail.com>
Cc: Ron Minnich <rminnich@gmail.com>
Cc: Latchesar Ionkov <lucho@ionkov.net>
Signed-off-by: default avatar"Eric W. Biederman" <ebiederm@xmission.com>
parent b4642556

fs/9p/v9fs.c

+25 −5
Original line number Diff line number Diff line
@@ -161,7 +161,13 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) 
				ret = r;

				continue;

			}

			v9ses->dfltuid = option;

			v9ses->dfltuid = make_kuid(current_user_ns(), option);

			if (!uid_valid(v9ses->dfltuid)) {

				p9_debug(P9_DEBUG_ERROR,

					 "uid field, but not a uid?\n");

				ret = -EINVAL;

				continue;

			}

			break;

		case Opt_dfltgid:

			r = match_int(&args[0], &option);
@@ -171,7 +177,13 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) 
				ret = r;

				continue;

			}

			v9ses->dfltgid = option;

			v9ses->dfltgid = make_kgid(current_user_ns(), option);

			if (!gid_valid(v9ses->dfltgid)) {

				p9_debug(P9_DEBUG_ERROR,

					 "gid field, but not a gid?\n");

				ret = -EINVAL;

				continue;

			}

			break;

		case Opt_afid:

			r = match_int(&args[0], &option);
@@ -248,8 +260,9 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) 
			else if (strcmp(s, "client") == 0) {

				v9ses->flags |= V9FS_ACCESS_CLIENT;

			} else {

				uid_t uid;

				v9ses->flags |= V9FS_ACCESS_SINGLE;

				v9ses->uid = simple_strtoul(s, &e, 10);

				uid = simple_strtoul(s, &e, 10);

				if (*e != '\0') {

					ret = -EINVAL;

					pr_info("Unknown access argument %s\n",
@@ -257,6 +270,13 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) 
					kfree(s);

					goto free_and_return;

				}

				v9ses->uid = make_kuid(current_user_ns(), uid);

				if (!uid_valid(v9ses->uid)) {

					ret = -EINVAL;

					pr_info("Uknown uid %s\n", s);

					kfree(s);

					goto free_and_return;

				}

			}



			kfree(s);
@@ -319,7 +339,7 @@ struct p9_fid *v9fs_session_init(struct v9fs_session_info *v9ses, 
	list_add(&v9ses->slist, &v9fs_sessionlist);

	spin_unlock(&v9fs_sessionlist_lock);



	v9ses->uid = ~0;

	v9ses->uid = INVALID_UID;

	v9ses->dfltuid = V9FS_DEFUID;

	v9ses->dfltgid = V9FS_DEFGID;
@@ -364,7 +384,7 @@ struct p9_fid *v9fs_session_init(struct v9fs_session_info *v9ses, 


		v9ses->flags &= ~V9FS_ACCESS_MASK;

		v9ses->flags |= V9FS_ACCESS_ANY;

		v9ses->uid = ~0;

		v9ses->uid = INVALID_UID;

	}

	if (!v9fs_proto_dotl(v9ses) ||

		!((v9ses->flags & V9FS_ACCESS_MASK) == V9FS_ACCESS_CLIENT)) {

fs/9p/v9fs.h

+5 −5
Original line number Diff line number Diff line
@@ -109,9 +109,9 @@ struct v9fs_session_info { 
	char *uname;		/* user name to mount as */

	char *aname;		/* name of remote hierarchy being mounted */

	unsigned int maxdata;	/* max data for client interface */

	unsigned int dfltuid;	/* default uid/muid for legacy support */

	unsigned int dfltgid;	/* default gid for legacy support */

	u32 uid;		/* if ACCESS_SINGLE, the uid that has access */

	kuid_t dfltuid;		/* default uid/muid for legacy support */

	kgid_t dfltgid;		/* default gid for legacy support */

	kuid_t uid;		/* if ACCESS_SINGLE, the uid that has access */

	struct p9_client *clnt;	/* 9p client */

	struct list_head slist; /* list of sessions registered with v9fs */

	struct backing_dev_info bdi;
@@ -165,8 +165,8 @@ extern struct inode *v9fs_inode_from_fid_dotl(struct v9fs_session_info *v9ses, 
#define V9FS_PORT	564

#define V9FS_DEFUSER	"nobody"

#define V9FS_DEFANAME	""

#define V9FS_DEFUID	(-2)

#define V9FS_DEFGID	(-2)

#define V9FS_DEFUID	KUIDT_INIT(-2)

#define V9FS_DEFGID	KGIDT_INIT(-2)



static inline struct v9fs_session_info *v9fs_inode2v9ses(struct inode *inode)

{

CRA Git | Maintained and supported by SUSTech CRA and CCSE