Unverified Commit 29a69055 authored by Wedson Almeida Filho's avatar Wedson Almeida Filho Committed by Christian Brauner
Browse files

reiserfs: move reiserfs_xattr_handlers to .rodata 



This makes it harder for accidental or malicious changes to
reiserfs_xattr_handlers at runtime.

Cc: reiserfs-devel@vger.kernel.org
Signed-off-by: default avatarWedson Almeida Filho <walmeida@microsoft.com>
Link: https://lore.kernel.org/r/20230930050033.41174-23-wedsonaf@gmail.com


Signed-off-by: default avatarChristian Brauner <brauner@kernel.org>
parent 2e9440ac

fs/reiserfs/reiserfs.h

+1 −1
Original line number Diff line number Diff line
@@ -1165,7 +1165,7 @@ static inline int bmap_would_wrap(unsigned bmap_nr) 
	return bmap_nr > ((1LL << 16) - 1);

}



extern const struct xattr_handler *reiserfs_xattr_handlers[];

extern const struct xattr_handler * const reiserfs_xattr_handlers[];



/*

 * this says about version of key of all items (but stat data) the

fs/reiserfs/xattr.c

+2 −2
Original line number Diff line number Diff line
@@ -780,7 +780,7 @@ static inline bool reiserfs_posix_acl_list(const char *name, 
}



/* This is the implementation for the xattr plugin infrastructure */

static inline bool reiserfs_xattr_list(const struct xattr_handler **handlers,

static inline bool reiserfs_xattr_list(const struct xattr_handler * const *handlers,

				       const char *name, struct dentry *dentry)

{

	if (handlers) {
@@ -911,7 +911,7 @@ static int create_privroot(struct dentry *dentry) { return 0; } 
#endif



/* Actual operations that are exported to VFS-land */

const struct xattr_handler *reiserfs_xattr_handlers[] = {

const struct xattr_handler * const reiserfs_xattr_handlers[] = {

#ifdef CONFIG_REISERFS_FS_XATTR

	&reiserfs_xattr_user_handler,

	&reiserfs_xattr_trusted_handler,