[-mm PATCH] v9fs: cleanup fd transport

[Eric Van Hensbergen]

[PATCH] v9fs: cleanup fd transport

Signed-off-by: Latchesar Ionkov <[EMAIL PROTECTED]>
Signed-off-by: Eric Van Hensbegren <[EMAIL PROTECTED]>

---
commit a1949213f1723a7b8bba8edfa118985460d31604
tree 40224cafbfb68543c60a8e0f04ae669cba2cedf7
parent 3f92b2539fe581ee9011d687fbd43cebb641465e
author Eric Van Hensbergen <[EMAIL PROTECTED]> Wed, 31 Aug 2005 16:02:42
-0500
committer Eric Van Hensbergen <[EMAIL PROTECTED]> Wed, 31 Aug 2005
16:02:42 -0500

 fs/9p/trans_fd.c |   42 +++++++++++++++++++++++++++++++++++++++---
 fs/9p/v9fs.c     |    5 -----
 2 files changed, 39 insertions(+), 8 deletions(-)

diff --git a/fs/9p/trans_fd.c b/fs/9p/trans_fd.c
--- a/fs/9p/trans_fd.c
+++ b/fs/9p/trans_fd.c
@@ -56,6 +56,9 @@ static int v9fs_fd_recv(struct v9fs_tran
 {
        struct v9fs_trans_fd *ts = trans ? trans->priv : NULL;
 
+       if (!trans || trans->status != Connected || !ts)
+               return -EIO;
+
        return kernel_read(ts->in_file, ts->in_file->f_pos, v, len);
 }
 
@@ -73,6 +76,9 @@ static int v9fs_fd_send(struct v9fs_tran
        mm_segment_t oldfs = get_fs();
        int ret = 0;
 
+       if (!trans || trans->status != Connected || !ts)
+               return -EIO;
+
        set_fs(get_ds());
        /* The cast to a user pointer is valid due to the set_fs() */
        ret = vfs_write(ts->out_file, (void __user *)v, len,
&ts->out_file->f_pos);
@@ -95,6 +101,11 @@ v9fs_fd_init(struct v9fs_session_info *v
        struct v9fs_trans_fd *ts = NULL;
        struct v9fs_transport *trans = v9ses->transport;
 
+       if((v9ses->wfdno == ~0) || (v9ses->rfdno == ~0)) {
+               printk(KERN_ERR "v9fs: Insufficient options for proto=fd\n");
+               return -ENOPROTOOPT;
+       }
+
        sema_init(&trans->writelock, 1);
        sema_init(&trans->readlock, 1);
 
@@ -103,11 +114,21 @@ v9fs_fd_init(struct v9fs_session_info *v
        if (!ts)
                return -ENOMEM;
 
-       trans->priv = ts;
-
        ts->in_file = fget( v9ses->rfdno );
        ts->out_file = fget( v9ses->wfdno );
 
+       if (!ts->in_file || !ts->out_file) {
+               if (ts->in_file)
+                       fput(ts->in_file);
+
+               if (ts->out_file)
+                       fput(ts->out_file);
+
+               kfree(ts);
+               return -EIO;
+       }
+
+       trans->priv = ts;
        trans->status = Connected;
 
        return 0;
@@ -122,7 +143,22 @@ v9fs_fd_init(struct v9fs_session_info *v
 
 static void v9fs_fd_close(struct v9fs_transport *trans)
 {
-       struct v9fs_trans_fd *ts = trans ? trans->priv : NULL;
+       struct v9fs_trans_fd *ts;
+
+       if (!trans) 
+               return;
+
+       trans->status = Disconnected;
+       ts = trans->priv;
+
+       if (!ts)
+               return;
+
+       if (ts->in_file)
+               fput(ts->in_file);
+
+       if (ts->out_file)
+               fput(ts->out_file);
 
        kfree(ts);
 }
diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c
--- a/fs/9p/v9fs.c
+++ b/fs/9p/v9fs.c
@@ -296,11 +296,6 @@ v9fs_session_init(struct v9fs_session_in
        case PROTO_FD:
                trans_proto = &v9fs_trans_fd;
                *v9ses->remotename = 0;
-               if((v9ses->wfdno == ~0) || (v9ses->rfdno == ~0)) {
-                       printk(KERN_ERR "v9fs: Insufficient options for 
proto=fd\n");
-                       retval = -ENOPROTOOPT;
-                       goto SessCleanUp;
-               }
                break;
        default:
                printk(KERN_ERR "v9fs: Bad mount protocol %d\n", v9ses->proto);


-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html