If security_inode_copy_up() fails, it should not set new_creds, so no need
for the cleanup (which would've Oops-ed anyway, due to old_creds being
NULL).
Signed-off-by: Miklos Szeredi <mszeredi@redhat.com>
};
err = security_inode_copy_up(c->dentry, &new_creds);
- temp = ERR_PTR(err);
if (err < 0)
- goto out;
+ return ERR_PTR(err);
if (new_creds)
old_creds = override_creds(new_creds);
temp = ovl_do_tmpfile(c->workdir, c->stat.mode);
else
temp = ovl_create_temp(c->workdir, &cattr);
-out:
+
if (new_creds) {
revert_creds(old_creds);
put_cred(new_creds);