Revert "UBUNTU: SAUCE: apparmor: fix reference count leak when securityfs_setup_d_ino...

BugLink: http://bugs.launchpad.net/bugs/1669611
This reverts commit 090d374d7300e7d59fb32f1707a5622acad1ab80.

Signed-off-by: Tim Gardner <tim.gardner@canonical.com>

diff --git a/security/apparmor/apparmorfs.c b/security/apparmor/apparmorfs.c
index ee0f998c9e3ff4636f8db5f1f2bedda8c7405ec6..5786cca3911a2b19b3217b283cb16aee32c664ce 100644 (file)
--- a/security/apparmor/apparmorfs.c
+++ b/security/apparmor/apparmorfs.c
@@ -1052,12 +1052,12 @@ static int ns_mkdir_op(struct inode *dir, struct dentry *dentry, umode_t mode)
        error = securityfs_pin_fs();
        inode_lock_nested(dir, I_MUTEX_PARENT);
        if (error)
-               goto out;
+               return error;
 
        error = __securityfs_setup_d_inode(dir, dentry, mode | S_IFDIR,  NULL,
                                           NULL, NULL);
        if (error)
-               goto out;
+               return error;
 
        ns = aa_create_ns(parent, ACCESS_ONCE(dentry->d_name.name), dentry);
        if (IS_ERR(ns)) {
@@ -1066,7 +1066,6 @@ static int ns_mkdir_op(struct inode *dir, struct dentry *dentry, umode_t mode)
        }
 
        aa_put_ns(ns);          /* list ref remains */
-out:
        aa_put_ns(parent);
 
        return error;