From: Eric Van Hensbergen <ericvh@gmail.com>
Date: Sat, 16 Jan 2010 01:01:10 +0000 (-0600)
Subject: net/9p: fail when user specifies a transport which we can't find
X-Git-Tag: v2.6.33-rc8~36^2~6
X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=349d3bb878d71978650a0634b5445af3c1cc1cd8;p=~shefty%2Frdma-dev.git

net/9p: fail when user specifies a transport which we can't find

If the user specifies a transport and we can't find it, we failed back
to the default trainsport silently.  This patch will make the code
complain more loudly and return an error code.

Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
---

diff --git a/net/9p/client.c b/net/9p/client.c
index 8af95b2dddd..90a2eb926d1 100644
--- a/net/9p/client.c
+++ b/net/9p/client.c
@@ -108,6 +108,13 @@ static int parse_opts(char *opts, struct p9_client *clnt)
 			break;
 		case Opt_trans:
 			clnt->trans_mod = v9fs_get_trans_by_name(&args[0]);
+			if(clnt->trans_mod == NULL) {
+				P9_DPRINTK(P9_DEBUG_ERROR,
+				   "Could not find request transport: %s\n",
+				   (char *) &args[0]);
+				ret = -EINVAL;
+				goto free_and_return;
+			}
 			break;
 		case Opt_legacy:
 			clnt->dotu = 0;
@@ -117,6 +124,7 @@ static int parse_opts(char *opts, struct p9_client *clnt)
 		}
 	}
 
+free_and_return:
 	kfree(options);
 	return ret;
 }