diff options
author | Eric Van Hensbergen <ericvh@ericvh-desktop.austin.ibm.com> | 2007-07-13 17:47:58 -0400 |
---|---|---|
committer | Eric Van Hensbergen <ericvh@ericvh-desktop.austin.ibm.com> | 2007-07-14 16:14:19 -0400 |
commit | 0af8887ebf4556a76680a61b0bb156d934702c63 (patch) | |
tree | e9d8b0fba3825aa085d815a3cd265e1a47ead76e /net/9p | |
parent | 9e2f6688c0b52882496aff576b009bc1f7eea0b8 (diff) |
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>
Diffstat (limited to 'net/9p')
-rw-r--r-- | net/9p/client.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/9p/client.c b/net/9p/client.c index bb2b8a3af196..cb170750337c 100644 --- a/net/9p/client.c +++ b/net/9p/client.c | |||
@@ -127,12 +127,12 @@ void p9_client_destroy(struct p9_client *clnt) | |||
127 | clnt->trans = NULL; | 127 | clnt->trans = NULL; |
128 | } | 128 | } |
129 | 129 | ||
130 | if (clnt->fidpool) | ||
131 | p9_idpool_destroy(clnt->fidpool); | ||
132 | |||
133 | list_for_each_entry_safe(fid, fidptr, &clnt->fidlist, flist) | 130 | list_for_each_entry_safe(fid, fidptr, &clnt->fidlist, flist) |
134 | p9_fid_destroy(fid); | 131 | p9_fid_destroy(fid); |
135 | 132 | ||
133 | if (clnt->fidpool) | ||
134 | p9_idpool_destroy(clnt->fidpool); | ||
135 | |||
136 | kfree(clnt); | 136 | kfree(clnt); |
137 | } | 137 | } |
138 | EXPORT_SYMBOL(p9_client_destroy); | 138 | EXPORT_SYMBOL(p9_client_destroy); |