xfs: remove unnecessary indenting from xfs_attr3_leaf_getvalue
authorDave Chinner <dchinner@redhat.com>
Thu, 29 Aug 2019 16:04:09 +0000 (09:04 -0700)
committerDarrick J. Wong <darrick.wong@oracle.com>
Sat, 31 Aug 2019 05:43:57 +0000 (22:43 -0700)
Signed-off-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
fs/xfs/libxfs/xfs_attr_leaf.c

index 7a9440b7ab00c1f1f3fef2a7464ffd6a4da11ff5..c7378bc62d2b7ff072657b477f6c680a565e7f6c 100644 (file)
@@ -2391,24 +2391,25 @@ xfs_attr3_leaf_getvalue(
                }
                args->valuelen = valuelen;
                memcpy(args->value, &name_loc->nameval[args->namelen], valuelen);
-       } else {
-               name_rmt = xfs_attr3_leaf_name_remote(leaf, args->index);
-               ASSERT(name_rmt->namelen == args->namelen);
-               ASSERT(memcmp(args->name, name_rmt->name, args->namelen) == 0);
-               args->rmtvaluelen = be32_to_cpu(name_rmt->valuelen);
-               args->rmtblkno = be32_to_cpu(name_rmt->valueblk);
-               args->rmtblkcnt = xfs_attr3_rmt_blocks(args->dp->i_mount,
-                                                      args->rmtvaluelen);
-               if (args->flags & ATTR_KERNOVAL) {
-                       args->valuelen = args->rmtvaluelen;
-                       return 0;
-               }
-               if (args->valuelen < args->rmtvaluelen) {
-                       args->valuelen = args->rmtvaluelen;
-                       return -ERANGE;
-               }
+               return 0;
+       }
+
+       name_rmt = xfs_attr3_leaf_name_remote(leaf, args->index);
+       ASSERT(name_rmt->namelen == args->namelen);
+       ASSERT(memcmp(args->name, name_rmt->name, args->namelen) == 0);
+       args->rmtvaluelen = be32_to_cpu(name_rmt->valuelen);
+       args->rmtblkno = be32_to_cpu(name_rmt->valueblk);
+       args->rmtblkcnt = xfs_attr3_rmt_blocks(args->dp->i_mount,
+                                              args->rmtvaluelen);
+       if (args->flags & ATTR_KERNOVAL) {
+               args->valuelen = args->rmtvaluelen;
+               return 0;
+       }
+       if (args->valuelen < args->rmtvaluelen) {
                args->valuelen = args->rmtvaluelen;
+               return -ERANGE;
        }
+       args->valuelen = args->rmtvaluelen;
        return 0;
 }