diff options
author | Greg Kroah-Hartman <gregkh@suse.de> | 2006-03-16 18:44:26 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2006-03-20 16:42:57 -0500 |
commit | 641e6f30a095f3752ed84fd9d279382f5d3ef4c1 (patch) | |
tree | 59b57c14f9249488ded6814de4512f7f9949289f /fs/sysfs/inode.c | |
parent | c4a1745aa09fc110afdefea0e5d025043e348bae (diff) |
[PATCH] sysfs: sysfs_remove_dir() needs to invalidate the dentry
When calling sysfs_remove_dir() don't allow any further sysfs functions
to work for this kobject anymore. This fixes a nasty USB cdc-acm oops
on disconnect.
Many thanks to Bob Copeland and Paul Fulghum for taking the time to
track this down.
Cc: Bob Copeland <email@bobcopeland.com>
Cc: Paul Fulghum <paulkf@microgate.com>
Cc: Maneesh Soni <maneesh@in.ibm.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs/sysfs/inode.c')
-rw-r--r-- | fs/sysfs/inode.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/sysfs/inode.c b/fs/sysfs/inode.c index 689f7bcfaf30..6beee6f6a674 100644 --- a/fs/sysfs/inode.c +++ b/fs/sysfs/inode.c | |||
@@ -227,12 +227,16 @@ void sysfs_drop_dentry(struct sysfs_dirent * sd, struct dentry * parent) | |||
227 | void sysfs_hash_and_remove(struct dentry * dir, const char * name) | 227 | void sysfs_hash_and_remove(struct dentry * dir, const char * name) |
228 | { | 228 | { |
229 | struct sysfs_dirent * sd; | 229 | struct sysfs_dirent * sd; |
230 | struct sysfs_dirent * parent_sd = dir->d_fsdata; | 230 | struct sysfs_dirent * parent_sd; |
231 | |||
232 | if (!dir) | ||
233 | return; | ||
231 | 234 | ||
232 | if (dir->d_inode == NULL) | 235 | if (dir->d_inode == NULL) |
233 | /* no inode means this hasn't been made visible yet */ | 236 | /* no inode means this hasn't been made visible yet */ |
234 | return; | 237 | return; |
235 | 238 | ||
239 | parent_sd = dir->d_fsdata; | ||
236 | mutex_lock(&dir->d_inode->i_mutex); | 240 | mutex_lock(&dir->d_inode->i_mutex); |
237 | list_for_each_entry(sd, &parent_sd->s_children, s_sibling) { | 241 | list_for_each_entry(sd, &parent_sd->s_children, s_sibling) { |
238 | if (!sd->s_element) | 242 | if (!sd->s_element) |