From 9b6fafbd4ff56eb6e0e386fa903deaf4e1327dee Mon Sep 17 00:00:00 2001 From: "Aneesh Kumar K.V" Date: Tue, 30 Aug 2011 12:19:34 +0530 Subject: [PATCH] --- yaml --- r: 267251 b: refs/heads/master c: 4d5077f1b2aa502a0ca98b450d1b16fbccfe9c63 h: refs/heads/master i: 267249: 2429cac71a4e75f78deecf0e2b3cc799d76a6040 267247: b381619efc842710688c48fe45dd719b8aa982fd v: v3 --- [refs] | 2 +- trunk/fs/9p/v9fs.c | 33 ++++++++++++++++++++++++++------- trunk/net/9p/client.c | 12 +++++------- 3 files changed, 32 insertions(+), 15 deletions(-) diff --git a/[refs] b/[refs] index b307b3783622..b47398b37a30 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: 5635fd0ccf4ce90a9ecf8fa48f5f031f5f73e6f5 +refs/heads/master: 4d5077f1b2aa502a0ca98b450d1b16fbccfe9c63 diff --git a/trunk/fs/9p/v9fs.c b/trunk/fs/9p/v9fs.c index ef9661886112..2b78014a124a 100644 --- a/trunk/fs/9p/v9fs.c +++ b/trunk/fs/9p/v9fs.c @@ -132,21 +132,19 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) options = tmp_options; while ((p = strsep(&options, ",")) != NULL) { - int token; + int token, r; if (!*p) continue; token = match_token(p, tokens, args); - if (token < Opt_uname) { - int r = match_int(&args[0], &option); + switch (token) { + case Opt_debug: + r = match_int(&args[0], &option); if (r < 0) { P9_DPRINTK(P9_DEBUG_ERROR, - "integer field, but no integer?\n"); + "integer field, but no integer?\n"); ret = r; continue; } - } - switch (token) { - case Opt_debug: v9ses->debug = option; #ifdef CONFIG_NET_9P_DEBUG p9_debug_level = option; @@ -154,12 +152,33 @@ static int v9fs_parse_options(struct v9fs_session_info *v9ses, char *opts) break; case Opt_dfltuid: + r = match_int(&args[0], &option); + if (r < 0) { + P9_DPRINTK(P9_DEBUG_ERROR, + "integer field, but no integer?\n"); + ret = r; + continue; + } v9ses->dfltuid = option; break; case Opt_dfltgid: + r = match_int(&args[0], &option); + if (r < 0) { + P9_DPRINTK(P9_DEBUG_ERROR, + "integer field, but no integer?\n"); + ret = r; + continue; + } v9ses->dfltgid = option; break; case Opt_afid: + r = match_int(&args[0], &option); + if (r < 0) { + P9_DPRINTK(P9_DEBUG_ERROR, + "integer field, but no integer?\n"); + ret = r; + continue; + } v9ses->afid = option; break; case Opt_uname: diff --git a/trunk/net/9p/client.c b/trunk/net/9p/client.c index 9eadadb0a698..0edee4de608a 100644 --- a/trunk/net/9p/client.c +++ b/trunk/net/9p/client.c @@ -123,21 +123,19 @@ static int parse_opts(char *opts, struct p9_client *clnt) options = tmp_options; while ((p = strsep(&options, ",")) != NULL) { - int token; + int token, r; if (!*p) continue; token = match_token(p, tokens, args); - if (token < Opt_trans) { - int r = match_int(&args[0], &option); + switch (token) { + case Opt_msize: + r = match_int(&args[0], &option); if (r < 0) { P9_DPRINTK(P9_DEBUG_ERROR, - "integer field, but no integer?\n"); + "integer field, but no integer?\n"); ret = r; continue; } - } - switch (token) { - case Opt_msize: clnt->msize = option; break; case Opt_trans: