summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2019-05-15 15:33:32 +0200
committerJan Kara <jack@suse.cz>2019-05-20 10:50:48 +0200
commit8cd0f2ba787a1a1758dbe063fe6a26ceccd37c24 (patch)
tree894574868df41d4582b6ff2ccc507b7963a2fef1
parentf4c3fb8c433f1818da46b720754a5d4a8525fdf0 (diff)
ext2: Merge loops in ext2_xattr_set()
There are two very similar loops when searching xattr to set. Just merge them. Reviewed-by: Chengguang Xu <cgxu519@zoho.com.cn> Signed-off-by: Jan Kara <jack@suse.cz>
-rw-r--r--fs/ext2/xattr.c41
1 files changed, 19 insertions, 22 deletions
diff --git a/fs/ext2/xattr.c b/fs/ext2/xattr.c
index fb2e008d4406..f9fda6d16d78 100644
--- a/fs/ext2/xattr.c
+++ b/fs/ext2/xattr.c
@@ -436,28 +436,12 @@ bad_block:
error = -EIO;
goto cleanup;
}
- /* Find the named attribute. */
- here = FIRST_ENTRY(bh);
- while (!IS_LAST_ENTRY(here)) {
- struct ext2_xattr_entry *next = EXT2_XATTR_NEXT(here);
- if ((char *)next >= end)
- goto bad_block;
- if (!here->e_value_block && here->e_value_size) {
- size_t offs = le16_to_cpu(here->e_value_offs);
- if (offs < min_offs)
- min_offs = offs;
- }
- not_found = name_index - here->e_name_index;
- if (!not_found)
- not_found = name_len - here->e_name_len;
- if (!not_found)
- not_found = memcmp(name, here->e_name,name_len);
- if (not_found <= 0)
- break;
- here = next;
- }
- last = here;
- /* We still need to compute min_offs and last. */
+ /*
+ * Find the named attribute. If not found, 'here' will point
+ * to entry where the new attribute should be inserted to
+ * maintain sorting.
+ */
+ last = FIRST_ENTRY(bh);
while (!IS_LAST_ENTRY(last)) {
struct ext2_xattr_entry *next = EXT2_XATTR_NEXT(last);
if ((char *)next >= end)
@@ -467,8 +451,21 @@ bad_block:
if (offs < min_offs)
min_offs = offs;
}
+ if (not_found > 0) {
+ not_found = name_index - last->e_name_index;
+ if (!not_found)
+ not_found = name_len - last->e_name_len;
+ if (!not_found) {
+ not_found = memcmp(name, last->e_name,
+ name_len);
+ }
+ if (not_found <= 0)
+ here = last;
+ }
last = next;
}
+ if (not_found > 0)
+ here = last;
/* Check whether we have enough space left. */
free = min_offs - ((char*)last - (char*)header) - sizeof(__u32);