diff options
Diffstat (limited to 'fs/configfs/symlink.c')
-rw-r--r-- | fs/configfs/symlink.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/fs/configfs/symlink.c b/fs/configfs/symlink.c index 676c84c416da..faeb4417a10d 100644 --- a/fs/configfs/symlink.c +++ b/fs/configfs/symlink.c | |||
@@ -77,12 +77,15 @@ static int create_link(struct config_item *parent_item, | |||
77 | sl = kmalloc(sizeof(struct configfs_symlink), GFP_KERNEL); | 77 | sl = kmalloc(sizeof(struct configfs_symlink), GFP_KERNEL); |
78 | if (sl) { | 78 | if (sl) { |
79 | sl->sl_target = config_item_get(item); | 79 | sl->sl_target = config_item_get(item); |
80 | /* FIXME: needs a lock, I'd bet */ | 80 | spin_lock(&configfs_dirent_lock); |
81 | list_add(&sl->sl_list, &target_sd->s_links); | 81 | list_add(&sl->sl_list, &target_sd->s_links); |
82 | spin_unlock(&configfs_dirent_lock); | ||
82 | ret = configfs_create_link(sl, parent_item->ci_dentry, | 83 | ret = configfs_create_link(sl, parent_item->ci_dentry, |
83 | dentry); | 84 | dentry); |
84 | if (ret) { | 85 | if (ret) { |
86 | spin_lock(&configfs_dirent_lock); | ||
85 | list_del_init(&sl->sl_list); | 87 | list_del_init(&sl->sl_list); |
88 | spin_unlock(&configfs_dirent_lock); | ||
86 | config_item_put(item); | 89 | config_item_put(item); |
87 | kfree(sl); | 90 | kfree(sl); |
88 | } | 91 | } |
@@ -186,8 +189,9 @@ int configfs_unlink(struct inode *dir, struct dentry *dentry) | |||
186 | type->ct_item_ops->drop_link(parent_item, | 189 | type->ct_item_ops->drop_link(parent_item, |
187 | sl->sl_target); | 190 | sl->sl_target); |
188 | 191 | ||
189 | /* FIXME: Needs lock */ | 192 | spin_lock(&configfs_dirent_lock); |
190 | list_del_init(&sl->sl_list); | 193 | list_del_init(&sl->sl_list); |
194 | spin_unlock(&configfs_dirent_lock); | ||
191 | 195 | ||
192 | /* Put reference from create_link() */ | 196 | /* Put reference from create_link() */ |
193 | config_item_put(sl->sl_target); | 197 | config_item_put(sl->sl_target); |