summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLouis Rilling <Louis.Rilling@kerlabs.com>2008-06-12 17:26:47 +0200
committerMark Fasheh <mfasheh@suse.com>2008-07-14 13:57:16 -0700
commite75206517504461778c283b942440ef312e437d5 (patch)
treea0acb0d8cc4527fc00560d7b78a971100fba7f57
parent11c3b79218390a139f2d474ee1e983a672d5839a (diff)
downloadop-kernel-dev-e75206517504461778c283b942440ef312e437d5.zip
op-kernel-dev-e75206517504461778c283b942440ef312e437d5.tar.gz
configfs: call drop_link() to cleanup after create_link() failure
When allow_link() succeeds but create_link() fails, the subsystem is not informed of the failure. This patch fixes this by calling drop_link() on create_link() failures. Signed-off-by: Louis Rilling <Louis.Rilling@kerlabs.com> Signed-off-by: Joel Becker <joel.becker@oracle.com>
-rw-r--r--fs/configfs/symlink.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/fs/configfs/symlink.c b/fs/configfs/symlink.c
index faeb441..0004d18 100644
--- a/fs/configfs/symlink.c
+++ b/fs/configfs/symlink.c
@@ -140,8 +140,12 @@ int configfs_symlink(struct inode *dir, struct dentry *dentry, const char *symna
goto out_put;
ret = type->ct_item_ops->allow_link(parent_item, target_item);
- if (!ret)
+ if (!ret) {
ret = create_link(parent_item, target_item, dentry);
+ if (ret && type->ct_item_ops->drop_link)
+ type->ct_item_ops->drop_link(parent_item,
+ target_item);
+ }
config_item_put(target_item);
path_put(&nd.path);
OpenPOWER on IntegriCloud