[PATCH 01/11] VFS: discard err2 in filename_create()

[NeilBrown]

During the time when "err2" holds a value, "error" does not hold any
meaningful value.  So there is no need for two different err variables.
This patch discards "err2" and uses "error' throughout.

Signed-off-by: NeilBrown <n...@brown.name>
---
 fs/namei.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/fs/namei.c b/fs/namei.c
index cd43ff89fbaa..62c1e2268942 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -4114,7 +4114,6 @@ static struct dentry *filename_create(int dfd, struct 
filename *name,
        unsigned int reval_flag = lookup_flags & LOOKUP_REVAL;
        unsigned int create_flags = LOOKUP_CREATE | LOOKUP_EXCL;
        int type;
-       int err2;
        int error;
 
        error = filename_parentat(dfd, name, reval_flag, path, &last, &type);
@@ -4129,7 +4128,7 @@ static struct dentry *filename_create(int dfd, struct 
filename *name,
                goto out;
 
        /* don't fail immediately if it's r/o, at least try to report other 
errors */
-       err2 = mnt_want_write(path->mnt);
+       error = mnt_want_write(path->mnt);
        /*
         * Do the final lookup.  Suppress 'create' if there is a trailing
         * '/', and a directory wasn't requested.
@@ -4142,17 +4141,16 @@ static struct dentry *filename_create(int dfd, struct 
filename *name,
        if (IS_ERR(dentry))
                goto unlock;
 
-       if (unlikely(err2)) {
-               error = err2;
+       if (unlikely(error))
                goto fail;
-       }
+
        return dentry;
 fail:
        dput(dentry);
        dentry = ERR_PTR(error);
 unlock:
        inode_unlock(path->dentry->d_inode);
-       if (!err2)
+       if (!error)
                mnt_drop_write(path->mnt);
 out:
        path_put(path);
-- 
2.50.0.107.gf914562f5916.dirty