commit 2c4f9923374843384cfd3a2edc0f02c19a7a29e2
author Eric Biggers <ebiggers@google.com> Sat Apr 29 23:56:52 2017 -0400
committer Theodore Ts'o <tytso@mit.edu> Sat Apr 29 23:56:52 2017 -0400
tree 14cb6c83a5674bb046b117b4f550a705daf61bc2
parent ba7ea1d8f45383cb88858057e9bf60cd8cf3b898

ext4: rename ext4_xattr_check_names() to ext4_xattr_check_entries()

ext4_xattr_check_names() actually validates both the xattr names and
values, not just the names.  So rename it to ext4_xattr_check_entries()
to avoid confusion.

Signed-off-by: Eric Biggers <ebiggers@google.com>
Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>

fs/ext4/xattr.c

diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c
index 8e9d0d4..c5a3e98 100644
--- a/fs/ext4/xattr.c
+++ b/fs/ext4/xattr.c
@@ -162,8 +162,8 @@
 }
 
 static int
-ext4_xattr_check_names(struct ext4_xattr_entry *entry, void *end,
-		       void *value_start)
+ext4_xattr_check_entries(struct ext4_xattr_entry *entry, void *end,
+			 void *value_start)
 {
 	struct ext4_xattr_entry *e = entry;
 
@@ -217,8 +217,8 @@
 		return -EFSCORRUPTED;
 	if (!ext4_xattr_block_csum_verify(inode, bh))
 		return -EFSBADCRC;
-	error = ext4_xattr_check_names(BFIRST(bh), bh->b_data + bh->b_size,
-				       bh->b_data);
+	error = ext4_xattr_check_entries(BFIRST(bh), bh->b_data + bh->b_size,
+					 bh->b_data);
 	if (!error)
 		set_buffer_verified(bh);
 	return error;
@@ -233,7 +233,7 @@
 	if (end - (void *)header < sizeof(*header) + sizeof(u32) ||
 	    (header->h_magic != cpu_to_le32(EXT4_XATTR_MAGIC)))
 		goto errout;
-	error = ext4_xattr_check_names(IFIRST(header), end, IFIRST(header));
+	error = ext4_xattr_check_entries(IFIRST(header), end, IFIRST(header));
 errout:
 	if (error)
 		__ext4_error_inode(inode, function, line, 0,