summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Van Hensbergen <ericvh@ericvh-desktop.austin.ibm.com>2007-07-13 16:47:58 -0500
committerEric Van Hensbergen <ericvh@ericvh-desktop.austin.ibm.com>2007-07-14 15:14:19 -0500
commit0af8887ebf4556a76680a61b0bb156d934702c63 (patch)
treee9d8b0fba3825aa085d815a3cd265e1a47ead76e
parent9e2f6688c0b52882496aff576b009bc1f7eea0b8 (diff)
downloadop-kernel-dev-0af8887ebf4556a76680a61b0bb156d934702c63.zip
op-kernel-dev-0af8887ebf4556a76680a61b0bb156d934702c63.tar.gz
9p: fix a race condition bug in umount which caused a segfault
umounting partitions after heavy activity would sometimes trigger a segmentation violation. This fix appears to remove that problem. Fix originally provided by Latchesar Ionkov. Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
-rw-r--r--net/9p/client.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/net/9p/client.c b/net/9p/client.c
index bb2b8a3..cb17075 100644
--- a/net/9p/client.c
+++ b/net/9p/client.c
@@ -127,12 +127,12 @@ void p9_client_destroy(struct p9_client *clnt)
clnt->trans = NULL;
}
- if (clnt->fidpool)
- p9_idpool_destroy(clnt->fidpool);
-
list_for_each_entry_safe(fid, fidptr, &clnt->fidlist, flist)
p9_fid_destroy(fid);
+ if (clnt->fidpool)
+ p9_idpool_destroy(clnt->fidpool);
+
kfree(clnt);
}
EXPORT_SYMBOL(p9_client_destroy);
OpenPOWER on IntegriCloud